Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 4: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/542/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: ASTERIXDB-1238: Refactor AqlMetadataProvider
Jenkins has posted comments on this change. Change subject: ASTERIXDB-1238: Refactor AqlMetadataProvider .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2498/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1016 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If2720817c5659622e1f713653856825d612eb892 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: ASTERIXDB-1238: Refactor AqlMetadataProvider
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1016 to look at the new patch set (#3). Change subject: ASTERIXDB-1238: Refactor AqlMetadataProvider .. ASTERIXDB-1238: Refactor AqlMetadataProvider Change-Id: If2720817c5659622e1f713653856825d612eb892 --- M asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/AqlMetadataProvider.java A asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataManagerUtil.java 2 files changed, 1,151 insertions(+), 1,602 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/16/1016/3 -- To view, visit https://asterix-gerrit.ics.uci.edu/1016 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If2720817c5659622e1f713653856825d612eb892 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-Reviewer: abdullah alamoudi
Change in asterixdb[master]: Start/Stop Sample Scripts for Windows
Jenkins has posted comments on this change. Change subject: Start/Stop Sample Scripts for Windows .. Patch Set 1: Integration-Tests-1 Integration Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/540/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/1142 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id6eee2156a5c6e40e0e864c2b8be2c08593e71b1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/539/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
[jira] [Created] (ASTERIXDB-1626) Multiple global aggregations result in sub-optimal plan
Yingyi Bu created ASTERIXDB-1626: Summary: Multiple global aggregations result in sub-optimal plan Key: ASTERIXDB-1626 URL: https://issues.apache.org/jira/browse/ASTERIXDB-1626 Project: Apache AsterixDB Issue Type: Bug Components: Optimizer Reporter: Yingyi Bu Assignee: Yingyi Bu query: {noformat} Select count(*), avg(I_ID) FROM items; {noformat} {noformat} istribute result [%0->$$24] -- DISTRIBUTE_RESULT |UNPARTITIONED| exchange -- ONE_TO_ONE_EXCHANGE |UNPARTITIONED| project ([$$24]) -- STREAM_PROJECT |UNPARTITIONED| assign [$$24] <- [function-call: asterix:closed-record-constructor, Args:[AString: {$1}, function-call: asterix:sql-count, Args:[%0->$$19], AString: {$2}, function-call: asterix:sql-avg, Args:[%0->$$23]]] -- ASSIGN |UNPARTITIONED| project ([$$19, $$23]) -- STREAM_PROJECT |UNPARTITIONED| subplan { aggregate [$$23] <- [function-call: asterix:listify, Args:[%0->$$22]] -- AGGREGATE |UNPARTITIONED| assign [$$22] <- [function-call: asterix:field-access-by-name, Args:[%0->$$27, AString: {I_ID}]] -- ASSIGN |UNPARTITIONED| assign [$$27] <- [function-call: asterix:field-access-by-index, Args:[%0->$$12, AInt32: {0}]] -- ASSIGN |UNPARTITIONED| unnest $$12 <- function-call: asterix:scan-collection, Args:[%0->$$15] -- UNNEST |UNPARTITIONED| nested tuple source -- NESTED_TUPLE_SOURCE |UNPARTITIONED| } -- SUBPLAN |UNPARTITIONED| project ([$$19, $$15]) -- STREAM_PROJECT |UNPARTITIONED| subplan { aggregate [$$19] <- [function-call: asterix:listify, Args:[%0->$$18]] -- AGGREGATE |UNPARTITIONED| unnest $$11 <- function-call: asterix:scan-collection, Args:[%0->$$15] -- UNNEST |UNPARTITIONED| nested tuple source -- NESTED_TUPLE_SOURCE |UNPARTITIONED| } -- SUBPLAN |UNPARTITIONED| assign [$$18] <- [AInt64: {1}] -- ASSIGN |UNPARTITIONED| aggregate [$$15] <- [function-call: asterix:listify, Args:[%0->$$10]] -- AGGREGATE |UNPARTITIONED| exchange -- RANDOM_MERGE_EXCHANGE |PARTITIONED| project ([$$10]) -- STREAM_PROJECT |PARTITIONED| assign [$$10] <- [function-call: asterix:closed-record-constructor, Args:[AString: {items}, %0->$$7]] -- ASSIGN |PARTITIONED| project ([$$7]) -- STREAM_PROJECT |PARTITIONED| exchange -- ONE_TO_ONE_EXCHANGE |PARTITIONED| data-scan []<-[$$25, $$7, $$26] <- CouchbaseAnalytics:items -- DATASOURCE_SCAN |PARTITIONED| exchange -- ONE_TO_ONE_EXCHANGE |PARTITIONED| empty-tuple-source -- EMPTY_TUPLE_SOURCE |PARTITIONED| {noformat} -- This message was sent by Atlassian JIRA (v6.3.4#6332)
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Michael Blow has posted comments on this change. Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1137/1/asterixdb/asterix-server/pom.xml File asterixdb/asterix-server/pom.xml: Line 179: 1.0.0-SNAPSHOT > It can be anything we want. I'd almost prefer for it to be a never-changing Not sure how realistic it is, especially given that we only release semi-annually but I was thinking of a case where we multiple versions are sharing a single local maven repo- that wouldn't work very well if we keep a constant snapshot version across releases. That was my thought. -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris HilleryGerrit-Reviewer: Chris Hillery Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: Yes
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 4: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2497/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1138 to look at the new patch set (#4). Change subject: Add functions: atan2, power, trunc, and uuid. .. Add functions: atan2, power, trunc, and uuid. Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/misc/uuid/uuid.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/atan2/atan2.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/power/power.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/numeric/trunc/trunc.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/results/misc/uuid/uuid.1.adm A asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/atan2/atan2.1.adm A asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/power/power.1.adm A asterixdb/asterix-app/src/test/resources/runtimets/results/numeric/trunc/trunc.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml M asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/util/FunctionMapUtil.java M asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/AsterixBuiltinFunctions.java A asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/NumericDoubleOutputFunctionTypeComputer.java A asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/NumericInt8OutputFunctionTypeComputer.java R asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/NumericRound2TypeComputer.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericATan2Descriptor.java M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericModuloDescriptor.java M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericRoundHalfToEven2Descriptor.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/NumericTruncDescriptor.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/UUIDDescriptor.java 20 files changed, 744 insertions(+), 170 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/38/1138/4 -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins
Change in asterixdb[master]: Add string function repeat and split.
Jenkins has posted comments on this change. Change subject: Add string function repeat and split. .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/541/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1141 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Chris Hillery has posted comments on this change. Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1137/1/asterixdb/asterix-server/pom.xml File asterixdb/asterix-server/pom.xml: Line 179: 1.0.0-SNAPSHOT > I think this should match the asterixdb version (i.e. ${project.version}) It can be anything we want. I'd almost prefer for it to be a never-changing constant, and 1.0.0 seems like a reasonable constant to pick. But if you think it makes sense to use the asterixdb version, that's fine too I guess. It means we need to update asterix-opt whenever we bump the asterixdb version, but I suppose we'll need to anyway. -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris HilleryGerrit-Reviewer: Chris Hillery Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: Yes
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 3: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2496/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
[jira] [Created] (ASTERIXDB-1625) NC2NC Messaging networking parameters should be configurable when using NCService
Michael Blow created ASTERIXDB-1625: --- Summary: NC2NC Messaging networking parameters should be configurable when using NCService Key: ASTERIXDB-1625 URL: https://issues.apache.org/jira/browse/ASTERIXDB-1625 Project: Apache AsterixDB Issue Type: Bug Components: Hyracks Core Reporter: Michael Blow Currently the NC2NC messaging port, address are not able to be configured for clusters using the NCService. Ini properties need to be defined for the following: -messaging-ip-address: IP Address to bind messaging listener (default: same as -address) -messaging-port: IP port to bind messaging listener (default: random port) -messaging-public-ip-address: Public IP Address to announce messaging listener (default: same as -messaging-ip-address) -messaging-public-port: Public IP port to announce messaging listener -- This message was sent by Atlassian JIRA (v6.3.4#6332)
Change in asterixdb[master]: Start/Stop Sample Scripts for Windows
Jenkins has posted comments on this change. Change subject: Start/Stop Sample Scripts for Windows .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/540/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1142 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id6eee2156a5c6e40e0e864c2b8be2c08593e71b1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Add string function repeat and split.
Jenkins has posted comments on this change. Change subject: Add string function repeat and split. .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2495/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1141 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/539/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Start/Stop Sample Scripts for Windows
Michael Blow has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1142 Change subject: Start/Stop Sample Scripts for Windows .. Start/Stop Sample Scripts for Windows - Fix windows binaries (ncservice, cc, nc, helper) - Add windows start/stop sample cluster scripts Change-Id: Id6eee2156a5c6e40e0e864c2b8be2c08593e71b1 --- M .gitignore M asterixdb/asterix-client-helper/pom.xml M asterixdb/asterix-client-helper/src/main/assembly/assembly.xml M asterixdb/asterix-server/pom.xml M asterixdb/asterix-server/src/main/assembly/binary-assembly.xml A asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat A asterixdb/asterix-server/src/main/samples/local/bin/stop-sample-cluster.bat 7 files changed, 174 insertions(+), 11 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/42/1142/1 diff --git a/.gitignore b/.gitignore index 11d149b..ba4e894 100644 --- a/.gitignore +++ b/.gitignore @@ -1,6 +1,5 @@ invIndex* primaryBTree* -bin target .classpath .settings @@ -13,8 +12,6 @@ build asterixdb/asterix-opt asterixdb/asterix-app/src/test/resources/externallib -asterixdb/asterix-app/rttest -asterixdb/asterix-app/mdtest asterixdb/asterix-app/opttest asterixdb/asterix-app/parserts asterixdb/asterix-app/library @@ -51,4 +48,4 @@ .DS_Store *.swp .m2* -ß \ No newline at end of file +ß diff --git a/asterixdb/asterix-client-helper/pom.xml b/asterixdb/asterix-client-helper/pom.xml index 7068cf6..e6d5684 100644 --- a/asterixdb/asterix-client-helper/pom.xml +++ b/asterixdb/asterix-client-helper/pom.xml @@ -29,7 +29,7 @@ org.codehaus.mojo appassembler-maven-plugin -1.3 +1.10 ${project.build.directory}/appassembler @@ -46,7 +46,15 @@ org.apache.asterix.clienthelper.AsterixHelper - + + + cbashelper + org.apache.asterix.clienthelper.AsterixHelper + +booter-windows + + + @@ -84,5 +92,10 @@ javax.servlet javax.servlet-api + + org.codehaus.mojo.appassembler + appassembler-booter + 1.10 + diff --git a/asterixdb/asterix-client-helper/src/main/assembly/assembly.xml b/asterixdb/asterix-client-helper/src/main/assembly/assembly.xml index 2e4e133..122ee38 100644 --- a/asterixdb/asterix-client-helper/src/main/assembly/assembly.xml +++ b/asterixdb/asterix-client-helper/src/main/assembly/assembly.xml @@ -34,5 +34,17 @@ bin 0755 + + target/generated-resources/appassembler/booter-windows/etc + etc + +*.xml + + + + target/generated-resources/appassembler/booter-windows/bin + bin + 0755 + diff --git a/asterixdb/asterix-server/pom.xml b/asterixdb/asterix-server/pom.xml index 269c219..0897145 100644 --- a/asterixdb/asterix-server/pom.xml +++ b/asterixdb/asterix-server/pom.xml @@ -44,7 +44,7 @@ org.codehaus.mojo appassembler-maven-plugin -1.3 +1.10 ${project.build.directory}/appassembler @@ -235,7 +235,7 @@ org.codehaus.mojo.appassembler appassembler-booter - 1.3.1 + 1.10 org.apache.asterix diff --git a/asterixdb/asterix-server/src/main/assembly/binary-assembly.xml b/asterixdb/asterix-server/src/main/assembly/binary-assembly.xml index 0c82f41..19f0034 100644 --- a/asterixdb/asterix-server/src/main/assembly/binary-assembly.xml +++ b/asterixdb/asterix-server/src/main/assembly/binary-assembly.xml @@ -69,10 +69,9 @@ target/generated-resources/appassembler/booter-windows/etc - bin + etc -asterixcc.xml -asterixnc.xml +*.xml diff --git a/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat b/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat new file mode 100644 index 000..4d10261 --- /dev/null +++ b/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat @@ -0,0 +1,83 @@ +@REM +@REM Licensed to the Apache Software Foundation (ASF) under one +@REM or more contributor license agreements. See the NOTICE file +@REM distributed with this work for additional information +@REM regarding copyright ownership. The ASF licenses this file +@REM to you under the Apache License, Version 2.0 (the +@REM "License"); you may not use this file except in compliance +@REM with the License. You may obtain a copy of the License at +@REM +@REM http://www.apache.org/licenses/LICENSE-2.0 +@REM +@REM Unless required by applicable law or agreed to in writing, +@REM software
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 2: Integration-Tests-1 Integration Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/538/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Add string function repeat and split.
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1141 to look at the new patch set (#2). Change subject: Add string function repeat and split. .. Add string function repeat and split. Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat/repeat.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat_error/repeat_error.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/split/split.3.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/results/string/repeat/repeat.1.adm A asterixdb/asterix-app/src/test/resources/runtimets/results/string/split/split.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml M asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/AsterixBuiltinFunctions.java R asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/StringIntToStringTypeComputer.java A asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/StringToInt64ListTypeComputer.java A asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/StringToStringListTypeComputer.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/StringRepeatDescriptor.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/StringSplitDescriptor.java M hyracks-fullstack/hyracks/hyracks-data/hyracks-data-std/src/main/java/org/apache/hyracks/data/std/primitive/UTF8StringPointable.java 14 files changed, 534 insertions(+), 17 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/41/1141/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1141 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins
Change in asterixdb[master]: Add string function repeat and split.
Jenkins has posted comments on this change. Change subject: Add string function repeat and split. .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2493/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1141 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Add string function repeat and split.
Jenkins has posted comments on this change. Change subject: Add string function repeat and split. .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2491/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1141 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Add string function repeat and split.
Yingyi Bu has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1141 Change subject: Add string function repeat and split. .. Add string function repeat and split. Change-Id: Ib9de5a59807d5ff51fa5d72444053f87cf8dd289 --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat/repeat.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat_error/repeat_error.1.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/split/split.3.query.sqlpp A asterixdb/asterix-app/src/test/resources/runtimets/results/string/repeat/repeat.1.adm A asterixdb/asterix-app/src/test/resources/runtimets/results/string/split/split.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml M asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/AsterixBuiltinFunctions.java R asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/StringIntToStringTypeComputer.java A asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/StringToInt64ListTypeComputer.java A asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/StringToStringListTypeComputer.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/StringRepeatDescriptor.java A asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/StringSplitDescriptor.java M hyracks-fullstack/hyracks/hyracks-data/hyracks-data-std/src/main/java/org/apache/hyracks/data/std/primitive/UTF8StringPointable.java 14 files changed, 529 insertions(+), 13 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/41/1141/1 diff --git a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java index a1746cc..27454e3 100644 --- a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java +++ b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java @@ -222,6 +222,8 @@ import org.apache.asterix.runtime.evaluators.functions.StringRegExpPositionWithFlagDescriptor; import org.apache.asterix.runtime.evaluators.functions.StringRegExpReplaceDescriptor; import org.apache.asterix.runtime.evaluators.functions.StringRegExpReplaceWithFlagsDescriptor; +import org.apache.asterix.runtime.evaluators.functions.StringRepeatDescriptor; +import org.apache.asterix.runtime.evaluators.functions.StringSplitDescriptor; import org.apache.asterix.runtime.evaluators.functions.StringStartsWithDescriptor; import org.apache.asterix.runtime.evaluators.functions.StringToCodePointDescriptor; import org.apache.asterix.runtime.evaluators.functions.StringTrim2Descriptor; @@ -513,6 +515,8 @@ functionsToInjectUnkownHandling.add(StringLTrim2Descriptor.FACTORY); functionsToInjectUnkownHandling.add(StringRTrim2Descriptor.FACTORY); functionsToInjectUnkownHandling.add(StringPositionDescriptor.FACTORY); +functionsToInjectUnkownHandling.add(StringRepeatDescriptor.FACTORY); +functionsToInjectUnkownHandling.add(StringSplitDescriptor.FACTORY); // Constructors functionsToInjectUnkownHandling.add(ABooleanConstructorDescriptor.FACTORY); diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat/repeat.1.query.sqlpp b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat/repeat.1.query.sqlpp new file mode 100644 index 000..17904c7 --- /dev/null +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/repeat/repeat.1.query.sqlpp @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ + +{ + "a": repeat(" new ", 2), + "b": repeat(" abcx ", 0), + "c": repeat("", 2), + "d": repeat(null, 2), + "e": repeat("asc", null), +
Change in asterixdb[master]: improve display of end-time for jobs in Hyracks console
Jenkins has posted comments on this change. Change subject: improve display of end-time for jobs in Hyracks console .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/537/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1140 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I80383dd0d35d0bbb83e00f56e094392d6d375a6d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: fix NPE in ValidateUtil
Jenkins has posted comments on this change. Change subject: fix NPE in ValidateUtil .. Patch Set 3: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/536/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1139 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f623441a3f5b72a9a0621d32d9d1a6aa5567d66 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Yingyi Bu Gerrit-HasComments: No
Change in asterixdb[master]: improve display of end-time for jobs in Hyracks console
Jenkins has posted comments on this change. Change subject: improve display of end-time for jobs in Hyracks console .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/537/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1140 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I80383dd0d35d0bbb83e00f56e094392d6d375a6d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: fix NPE in ValidateUtil
Jenkins has posted comments on this change. Change subject: fix NPE in ValidateUtil .. Patch Set 3: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/536/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1139 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f623441a3f5b72a9a0621d32d9d1a6aa5567d66 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Yingyi Bu Gerrit-HasComments: No
Change in asterixdb[master]: set HTTP status code before writing the result
Till Westmann has submitted this change and it was merged. Change subject: set HTTP status code before writing the result .. set HTTP status code before writing the result Change-Id: I46adb4aeaaa1ada6669b7535bb6d0879a26bc319 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1134 Sonar-Qube: JenkinsTested-by: Jenkins Integration-Tests: Jenkins Reviewed-by: Michael Blow --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 2 files changed, 17 insertions(+), 13 deletions(-) Approvals: Michael Blow: Looks good to me, approved Jenkins: Verified; No violations found; Verified diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java index b4097d1..3b69dd1 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java @@ -342,7 +342,7 @@ printField(pw, ErrorField.CODE.str(), "1"); final String msg = rootCause.getMessage(); printField(pw, ErrorField.MSG.str(), -JSONUtil.escape(rootCause.getClass().getName() + ": " + msg != null ? msg : ""), addStack); +JSONUtil.escape(rootCause.getClass().getName() + ": " + (msg != null ? msg : "")), addStack); if (addStack) { StringWriter sw = new StringWriter(); PrintWriter stackWriter = new PrintWriter(sw); @@ -453,10 +453,10 @@ GlobalConfig.ASTERIX_LOGGER.log(Level.SEVERE, result); +response.setStatus(respCode); response.getWriter().print(result); if (response.getWriter().checkError()) { LOGGER.warning("Error flushing output writer"); } -response.setStatus(respCode); } } diff --git a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java index 3e16802..52237d8 100644 --- a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java +++ b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java @@ -250,21 +250,25 @@ } } +protected HttpResponse executeAndCheckHttpRequest(HttpUriRequest method) throws Exception { +return checkResponse(executeHttpRequest(method)); +} + protected HttpResponse executeHttpRequest(HttpUriRequest method) throws Exception { HttpClient client = HttpClients.custom() .setRetryHandler(StandardHttpRequestRetryHandler.INSTANCE) .build(); -HttpResponse httpResponse; - try { -httpResponse = client.execute(method); +return client.execute(method); } catch (Exception e) { GlobalConfig.ASTERIX_LOGGER.log(Level.SEVERE, e.getMessage(), e); e.printStackTrace(); throw e; } -int statusCode = httpResponse.getStatusLine().getStatusCode(); -if (statusCode != HttpStatus.SC_OK) { +} + +protected HttpResponse checkResponse(HttpResponse httpResponse) throws Exception { +if (httpResponse.getStatusLine().getStatusCode() != HttpStatus.SC_OK) { // QQQ For now, we are indeed assuming we get back JSON errors. // In future this may be changed depending on the requested // output format sent to the servlet. @@ -294,7 +298,7 @@ HttpUriRequest method = constructHttpMethod(str, url, "query", false, params); // Set accepted output response type method.setHeader("Accept", fmt.mimeType()); -HttpResponse response = executeHttpRequest(method); +HttpResponse response = executeAndCheckHttpRequest(method); return response.getEntity().getContent(); } @@ -368,7 +372,7 @@ public InputStream executeClusterStateQuery(OutputFormat fmt, String url) throws Exception { HttpUriRequest request = RequestBuilder.get(url).setHeader("Accept", fmt.mimeType()).build(); -HttpResponse response = executeHttpRequest(request); +HttpResponse response = executeAndCheckHttpRequest(request); return response.getEntity().getContent(); } @@ -381,7 +385,7 @@ .build(); // Execute the method. -executeHttpRequest(request); +executeAndCheckHttpRequest(request); } // Executes AQL in either async or async-defer mode. @@ -393,7 +397,7 @@
Change in asterixdb[master]: improve display of end-time for jobs in Hyracks console
Jenkins has posted comments on this change. Change subject: improve display of end-time for jobs in Hyracks console .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2490/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1140 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I80383dd0d35d0bbb83e00f56e094392d6d375a6d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: improve display of end-time for jobs in Hyracks console
Till Westmann has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1140 Change subject: improve display of end-time for jobs in Hyracks console .. improve display of end-time for jobs in Hyracks console Change-Id: I80383dd0d35d0bbb83e00f56e094392d6d375a6d --- M hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.java M hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/resources/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.html 2 files changed, 5 insertions(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/40/1140/1 diff --git a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.java b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.java index 709e406..32ac3f5 100644 --- a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.java +++ b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.java @@ -92,7 +92,10 @@ } private String longToDateString(long milliseconds) { -SimpleDateFormat sdf = new SimpleDateFormat("MMM dd, HH:mm:ss"); +if (milliseconds == 0) { +return "n/a"; +} +SimpleDateFormat sdf = new SimpleDateFormat("MMM dd, HH:mm:ss"); Date date = new Date(milliseconds); return sdf.format(date); } diff --git a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/resources/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.html b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/resources/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.html index bdba2cb..bbb22b7 100644 --- a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/resources/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.html +++ b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/resources/org/apache/hyracks/control/cc/adminconsole/pages/IndexPage.html @@ -49,6 +49,7 @@ Status Create Time Start Time +End Time Job Details -- To view, visit https://asterix-gerrit.ics.uci.edu/1140 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I80383dd0d35d0bbb83e00f56e094392d6d375a6d Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till Westmann
Change in asterixdb[master]: set HTTP status code before writing the result
Michael Blow has posted comments on this change. Change subject: set HTTP status code before writing the result .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1134 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I46adb4aeaaa1ada6669b7535bb6d0879a26bc319 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
[jira] [Commented] (ASTERIXDB-1624) Storage introspection APIs
[ https://issues.apache.org/jira/browse/ASTERIXDB-1624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15456506#comment-15456506 ] Till commented on ASTERIXDB-1624: - For the introspection, it would be nice to provide some built-in functions that return information as ADM instances. > Storage introspection APIs > -- > > Key: ASTERIXDB-1624 > URL: https://issues.apache.org/jira/browse/ASTERIXDB-1624 > Project: Apache AsterixDB > Issue Type: New Feature > Components: AsterixDB, Hyracks, Storage >Reporter: Michael J. Carey >Assignee: Ian Maxon > > It would be good to have more DDLs that can introspect about the storage of > datasets within an instance. Both at the level of the Hyracks storage > (whether or not a merge is going on, merge thresholds, number of components, > etc) as well as the AsterixDB level (NodeGroups, indices of datasets, etc). -- This message was sent by Atlassian JIRA (v6.3.4#6332)
Change in asterixdb[master]: set HTTP status code before writing the result
Jenkins has posted comments on this change. Change subject: set HTTP status code before writing the result .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/535/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1134 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I46adb4aeaaa1ada6669b7535bb6d0879a26bc319 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[master]: fix NPE in ValidateUtil
Jenkins has posted comments on this change. Change subject: fix NPE in ValidateUtil .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2488/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1139 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f623441a3f5b72a9a0621d32d9d1a6aa5567d66 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Yingyi Bu Gerrit-HasComments: No
Change in asterixdb[master]: fix NPE in ValidateUtil
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1139 to look at the new patch set (#2). Change subject: fix NPE in ValidateUtil .. fix NPE in ValidateUtil Change-Id: I8f623441a3f5b72a9a0621d32d9d1a6aa5567d66 --- M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/ValidateUtil.java M asterixdb/asterix-app/src/test/resources/runtimets/testsuite.xml M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml 3 files changed, 11 insertions(+), 7 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/39/1139/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1139 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I8f623441a3f5b72a9a0621d32d9d1a6aa5567d66 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Yingyi Bu
Change in asterixdb[master]: fix NPE in ValidateUtil
Jenkins has posted comments on this change. Change subject: fix NPE in ValidateUtil .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2487/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1139 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f623441a3f5b72a9a0621d32d9d1a6aa5567d66 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: set HTTP status code before writing the result
Jenkins has posted comments on this change. Change subject: set HTTP status code before writing the result .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2486/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1134 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I46adb4aeaaa1ada6669b7535bb6d0879a26bc319 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: No
Change in asterixdb[master]: set HTTP status code before writing the result
Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/1134 to look at the new patch set (#2). Change subject: set HTTP status code before writing the result .. set HTTP status code before writing the result Change-Id: I46adb4aeaaa1ada6669b7535bb6d0879a26bc319 --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/aql/TestExecutor.java 2 files changed, 17 insertions(+), 13 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/34/1134/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1134 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I46adb4aeaaa1ada6669b7535bb6d0879a26bc319 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till WestmannGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow
Change in asterixdb[master]: Add functions: atan2, power, trunc, and uuid.
Jenkins has posted comments on this change. Change subject: Add functions: atan2, power, trunc, and uuid. .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2485/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1138 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3aa3be97809be7d03bbab919252af7bbfe6f00e0 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi BuGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Output From ShutdownAPIServlet; status 405 on GET
Michael Blow has submitted this change and it was merged. Change subject: Output From ShutdownAPIServlet; status 405 on GET .. Output From ShutdownAPIServlet; status 405 on GET Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1136 Sonar-Qube: JenkinsTested-by: Jenkins Integration-Tests: Jenkins Reviewed-by: Till Westmann --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ShutdownAPIServlet.java 1 file changed, 22 insertions(+), 10 deletions(-) Approvals: Till Westmann: Looks good to me, approved Jenkins: Verified; No violations found; Verified diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ShutdownAPIServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ShutdownAPIServlet.java index 6d8c444..dd2f799 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ShutdownAPIServlet.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ShutdownAPIServlet.java @@ -19,6 +19,8 @@ package org.apache.asterix.api.http.servlet; import java.io.IOException; +import java.io.PrintWriter; +import java.util.Date; import java.util.logging.Level; import javax.servlet.ServletContext; @@ -28,7 +30,9 @@ import javax.servlet.http.HttpServletResponse; import org.apache.asterix.common.config.GlobalConfig; +import org.apache.asterix.om.util.AsterixClusterProperties; import org.apache.hyracks.api.client.IHyracksClientConnection; +import org.json.JSONObject; import static org.apache.asterix.api.http.servlet.ServletConstants.HYRACKS_CONNECTION_ATTR; @@ -39,23 +43,31 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { -response.setContentType("application/json"); -response.setCharacterEncoding("utf-8"); - ServletContext context = getServletContext(); IHyracksClientConnection hcc = (IHyracksClientConnection) context.getAttribute(HYRACKS_CONNECTION_ATTR); Thread t = new Thread(() -> { try { hcc.stopCluster(); } catch (Exception e) { -GlobalConfig.ASTERIX_LOGGER.log(Level.SEVERE, e.getMessage(), e); +GlobalConfig.ASTERIX_LOGGER.log(Level.SEVERE, "Exception stopping cluster", e); } -}); -t.start(); -response.setStatus(HttpServletResponse.SC_ACCEPTED); -} +}, "Shutdown Servlet Worker"); -@Override -public void doGet(HttpServletRequest request, HttpServletResponse response) throws IOException { +response.setContentType("application/json"); +response.setCharacterEncoding("utf-8"); +response.setStatus(HttpServletResponse.SC_ACCEPTED); +JSONObject jsonObject = new JSONObject(); +try { +jsonObject.put("status", "SHUTTING_DOWN"); +jsonObject.put("date", new Date()); +jsonObject.put("cluster" , AsterixClusterProperties.INSTANCE.getClusterStateDescription()); + +final PrintWriter writer = response.getWriter(); +writer.print(jsonObject.toString(4)); +writer.close(); +} catch (Exception e) { +GlobalConfig.ASTERIX_LOGGER.log(Level.INFO, "Exception writing response", e); +} +t.start(); } } -- To view, visit https://asterix-gerrit.ics.uci.edu/1136 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann
Change in asterixdb[master]: Output From ShutdownAPIServlet; status 405 on GET
Till Westmann has posted comments on this change. Change subject: Output From ShutdownAPIServlet; status 405 on GET .. Patch Set 1: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/1136 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Michael Blow has posted comments on this change. Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1137/1/asterixdb/asterix-server/pom.xml File asterixdb/asterix-server/pom.xml: Line 179: 1.0.0-SNAPSHOT I think this should match the asterixdb version (i.e. ${project.version}) -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris HilleryGerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-HasComments: Yes
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Jenkins has posted comments on this change. Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/533/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris HilleryGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin.
Wenhai Li has posted comments on this change. Change subject: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin. .. Patch Set 7: @Chen Ok, I will give an example with comments ASAP. -- To view, visit https://asterix-gerrit.ics.uci.edu/1119 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1aef69a2278853fd9f8020da6639331b367ed5ad Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wenhai LiGerrit-Reviewer: Chen Li Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Jianfeng Jia Gerrit-Reviewer: Taewoo Kim Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Wenhai Li Gerrit-Reviewer: Yingyi Bu Gerrit-Reviewer: Yingyi Bu Gerrit-HasComments: No
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Jenkins has posted comments on this change. Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/533/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris HilleryGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Jenkins has posted comments on this change. Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2484/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris HilleryGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: Depend on asterix-opt-bom 1.0.0-SNAPSHOT
Chris Hillery has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/1137 Change subject: Depend on asterix-opt-bom 1.0.0-SNAPSHOT .. Depend on asterix-opt-bom 1.0.0-SNAPSHOT Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 --- M asterixdb/asterix-server/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/37/1137/1 diff --git a/asterixdb/asterix-server/pom.xml b/asterixdb/asterix-server/pom.xml index 269c219..c713505 100644 --- a/asterixdb/asterix-server/pom.xml +++ b/asterixdb/asterix-server/pom.xml @@ -176,7 +176,7 @@ org.apache.asterix asterix-opt-bom - 0.1-SNAPSHOT + 1.0.0-SNAPSHOT pom -- To view, visit https://asterix-gerrit.ics.uci.edu/1137 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ifd5b95bc052c87419d0e925e46f7741b97cc1132 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Chris Hillery
Change in asterixdb[master]: Output From ShutdownAPIServlet; status 405 on GET
Jenkins has posted comments on this change. Change subject: Output From ShutdownAPIServlet; status 405 on GET .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/532/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/1136 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Output From ShutdownAPIServlet; status 405 on GET
Jenkins has posted comments on this change. Change subject: Output From ShutdownAPIServlet; status 405 on GET .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/532/ -- To view, visit https://asterix-gerrit.ics.uci.edu/1136 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Output From ShutdownAPIServlet; status 405 on GET
Till Westmann has posted comments on this change. Change subject: Output From ShutdownAPIServlet; status 405 on GET .. Patch Set 1: TRIGGER asterix-gerrit-integration-tests -- To view, visit https://asterix-gerrit.ics.uci.edu/1136 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Output From ShutdownAPIServlet; status 405 on GET
Jenkins has posted comments on this change. Change subject: Output From ShutdownAPIServlet; status 405 on GET .. Patch Set 1: Integration-Tests-1 Integration Tests Timed Out https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/531/ : ABORTED -- To view, visit https://asterix-gerrit.ics.uci.edu/1136 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f1a7d707ebdcb2a6502bc06d35d7ed38121e472 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael BlowGerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: Applied the multiway fuzzyjoin based on the prefix-based joi...
Chen Li has posted comments on this change. Change subject: Applied the multiway fuzzyjoin based on the prefix-based join and the selectFuzzyJoin testCases. .. Patch Set 4: @Wenhai: I want to take a closer look at this patch. To help me and other reviewers, I wonder: - Can you add high-level comments in the patch to explain the main changes you are making or the main bugs you are fixing? - For those major changes, can we add brief comments to explain them? - This patch looks big. Is it possible to divide it to multiple, smaller patches? -- To view, visit https://asterix-gerrit.ics.uci.edu/1076 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8736f104905eeda763d39709e002c2b9629278cc Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wenhai LiGerrit-Reviewer: Chen Li Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Taewoo Kim Gerrit-Reviewer: Wenhai Li Gerrit-HasComments: No
Change in asterixdb[master]: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin.
Chen Li has posted comments on this change. Change subject: ASTERIX-1487: fix the wrong plan for inverted fuzzyjoin. .. Patch Set 7: Wenhai: I am trying to review this patch. To help me (or other reviewers), can you add a high-level description of the cause of the problem, and how you fix it? In addition, for those major changes (mainly the java file), add a few comments to explain those changes. That will help a lot! -- To view, visit https://asterix-gerrit.ics.uci.edu/1119 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1aef69a2278853fd9f8020da6639331b367ed5ad Gerrit-PatchSet: 7 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wenhai LiGerrit-Reviewer: Chen Li Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Jianfeng Jia Gerrit-Reviewer: Taewoo Kim Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Yingyi Bu Gerrit-Reviewer: Yingyi Bu Gerrit-HasComments: No