Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Yingyi Bu (Code Review)
Yingyi Bu has submitted this change and it was merged.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Remove unused methods in DatasetUtil, IndexUtil, and MetadataProvider.

Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1741
Sonar-Qube: Jenkins 
Reviewed-by: Till Westmann 
Integration-Tests: Jenkins 
Tested-by: Jenkins 
---
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/DatasetUtil.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/IndexUtil.java
3 files changed, 1 insertion(+), 133 deletions(-)

Approvals:
  Till Westmann: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified

Objections:
  Jenkins: Violations found



diff --git 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
index 5934f5e..750f89b 100644
--- 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
+++ 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
@@ -35,7 +35,6 @@
 import 
org.apache.asterix.common.dataflow.LSMTreeInsertDeleteOperatorDescriptor;
 import org.apache.asterix.common.exceptions.AsterixException;
 import org.apache.asterix.common.exceptions.CompilationException;
-import org.apache.asterix.common.library.ILibraryManager;
 import org.apache.asterix.common.transactions.JobId;
 import org.apache.asterix.common.utils.StoragePathUtil;
 import org.apache.asterix.external.adapter.factory.LookupAdapterFactory;
@@ -127,9 +126,7 @@
 import org.apache.hyracks.dataflow.std.file.IFileSplitProvider;
 import org.apache.hyracks.dataflow.std.result.ResultWriterOperatorDescriptor;
 import 
org.apache.hyracks.storage.am.btree.dataflow.BTreeSearchOperatorDescriptor;
-import org.apache.hyracks.storage.am.common.api.IMetadataPageManagerFactory;
 import 
org.apache.hyracks.storage.am.common.api.IModificationOperationCallbackFactory;
-import org.apache.hyracks.storage.am.common.api.IPrimitiveValueProviderFactory;
 import 
org.apache.hyracks.storage.am.common.api.ISearchOperationCallbackFactory;
 import 
org.apache.hyracks.storage.am.common.dataflow.IIndexDataflowHelperFactory;
 import 
org.apache.hyracks.storage.am.common.dataflow.IndexDataflowHelperFactory;
@@ -144,10 +141,7 @@
 
 private final ICcApplicationContext appCtx;
 private final IStorageComponentProvider storaegComponentProvider;
-private final IMetadataPageManagerFactory metadataPageManagerFactory;
-private final IPrimitiveValueProviderFactory primitiveValueProviderFactory;
 private final StorageProperties storageProperties;
-private final ILibraryManager libraryManager;
 private final Dataverse defaultDataverse;
 private final LockList locks;
 
@@ -170,9 +164,6 @@
 this.defaultDataverse = defaultDataverse;
 this.storaegComponentProvider = componentProvider;
 storageProperties = appCtx.getStorageProperties();
-libraryManager = appCtx.getLibraryManager();
-metadataPageManagerFactory = 
componentProvider.getMetadataPageManagerFactory();
-primitiveValueProviderFactory = 
componentProvider.getPrimitiveValueProviderFactory();
 locks = new LockList();
 }
 
@@ -195,10 +186,6 @@
 @Override
 public Map getConfig() {
 return config;
-}
-
-public ILibraryManager getLibraryManager() {
-return libraryManager;
 }
 
 public void setJobId(JobId jobId) {
@@ -259,10 +246,6 @@
 
 public void 
setResultSerializerFactoryProvider(IResultSerializerFactoryProvider rafp) {
 this.resultSerializerFactoryProvider = rafp;
-}
-
-public IResultSerializerFactoryProvider 
getResultSerializerFactoryProvider() {
-return resultSerializerFactoryProvider;
 }
 
 public boolean isWriteTransaction() {
@@ -379,10 +362,6 @@
 } catch (AsterixException e) {
 throw new AlgebricksException(e);
 }
-}
-
-public static AlgebricksAbsolutePartitionConstraint 
determineLocationConstraint(FeedDataSource feedDataSource) {
-return new 
AlgebricksAbsolutePartitionConstraint(feedDataSource.getLocations());
 }
 
 protected Pair 
buildLoadableDatasetScan(
diff --git 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/DatasetUtil.java
 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/DatasetUtil.java
index e2cfb8b..098645e 100644
--- 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/DatasetUtil.java
+++ 
b/asterixdb/asterix-metadata/src/main

Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2: BAD-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1158/ : FAILURE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1158/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5279/ (6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2792/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1: BAD-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1157/ : FAILURE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1157/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2792/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: SimpleDateFormat Is Not Thread Safe

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: SimpleDateFormat Is Not Thread Safe
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2791/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1740
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4e057e893269b71b157b154721d2de1fe1f8c020
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/412/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2790/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: SimpleDateFormat Is Not Thread Safe

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: SimpleDateFormat Is Not Thread Safe
..


Patch Set 1:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2791/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1740
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4e057e893269b71b157b154721d2de1fe1f8c020
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2789/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/336/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3800/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/596/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Michael Blow (Code Review)
Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1747

Change subject: Handle Spaces in Path For Sample Cluster
..

Handle Spaces in Path For Sample Cluster

Fixes ASTERIXDB-1842- start-sample-cluster.sh with user name ending with
space.

Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
---
M asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh
M asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh
M 
asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/SampleLocalClusterIT.java
3 files changed, 18 insertions(+), 17 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/47/1747/1

diff --git 
a/asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh 
b/asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh
index 4db0770..f7e0e34 100755
--- a/asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh
+++ b/asterixdb/asterix-server/src/main/opt/local/bin/start-sample-cluster.sh
@@ -48,23 +48,23 @@
   "$JAVA_HOME/bin/java" -version
   exit 2
 }
-DIRNAME=$(dirname $0)
+DIRNAME=$(dirname "$0")
 [ $(echo $DIRNAME | wc -l) -ne 1 ] && {
   echo "Paths with spaces are not supported"
   exit 3
 }
 
-CLUSTERDIR=$(cd $DIRNAME/..; echo $PWD)
-INSTALLDIR=$(cd $CLUSTERDIR/../..; echo $PWD)
+CLUSTERDIR=$(cd "$DIRNAME/.."; echo $PWD)
+INSTALLDIR=$(cd "$CLUSTERDIR/../.."; echo $PWD)
 LOGSDIR=$CLUSTERDIR/logs
 
 echo "CLUSTERDIR=$CLUSTERDIR"
 echo "INSTALLDIR=$INSTALLDIR"
 echo "LOGSDIR=$LOGSDIR"
 echo
-cd $CLUSTERDIR
-mkdir -p $LOGSDIR
-$INSTALLDIR/bin/${HELPER_COMMAND} get_cluster_state -quiet \
+cd "$CLUSTERDIR"
+mkdir -p "$LOGSDIR"
+"$INSTALLDIR/bin/${HELPER_COMMAND}" get_cluster_state -quiet \
 && echo "ERROR: sample cluster address (localhost:${LISTEN_PORT}) already 
in use" && exit 1
 
 if ps -ef | grep 
'java.*org\.apache\.hyracks\.control\.[cn]c\.\([CN]CDriver\|service\.NCService\)'
 > /tmp/$$_pids; then
@@ -91,10 +91,10 @@
   echo "--"
   date
   echo "--"
-) | tee -a $LOGSDIR/blue-service.log | tee -a $LOGSDIR/red-service.log >> 
$LOGSDIR/cc.log
+) | tee -a "$LOGSDIR/blue-service.log" | tee -a "$LOGSDIR/red-service.log" >> 
"$LOGSDIR/cc.log"
 echo "INFO: Starting sample cluster..."
-$INSTALLDIR/bin/${NC_SERVICE_COMMAND} -logdir - -config-file 
$CLUSTERDIR/conf/blue.conf >> $LOGSDIR/blue-service.log 2>&1 &
-$INSTALLDIR/bin/${NC_SERVICE_COMMAND} -logdir - >> $LOGSDIR/red-service.log 
2>&1 &
-$INSTALLDIR/bin/${CC_COMMAND} -config-file $CLUSTERDIR/conf/cc.conf >> 
$LOGSDIR/cc.log 2>&1 &
-$INSTALLDIR/bin/${HELPER_COMMAND} wait_for_cluster -timeout 30
+"$INSTALLDIR/bin/${NC_SERVICE_COMMAND}" -logdir - -config-file 
"$CLUSTERDIR/conf/blue.conf" >> "$LOGSDIR/blue-service.log" 2>&1 &
+"$INSTALLDIR/bin/${NC_SERVICE_COMMAND}" -logdir - >> 
"$LOGSDIR/red-service.log" 2>&1 &
+"$INSTALLDIR/bin/${CC_COMMAND}" -config-file "$CLUSTERDIR/conf/cc.conf" >> 
"$LOGSDIR/cc.log" 2>&1 &
+"$INSTALLDIR/bin/${HELPER_COMMAND}" wait_for_cluster -timeout 30
 exit $?
diff --git 
a/asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh 
b/asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh
index 4ba5789..dbce377 100755
--- a/asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh
+++ b/asterixdb/asterix-server/src/main/opt/local/bin/stop-sample-cluster.sh
@@ -48,17 +48,17 @@
   "$JAVA_HOME/bin/java" -version
   exit 2
 }
-DIRNAME=$(dirname $0)
+DIRNAME=$(dirname "$0")
 [ $(echo $DIRNAME | wc -l) -ne 1 ] && {
   echo "Paths with spaces are not supported"
   exit 3
 }
 
-CLUSTERDIR=$(cd $DIRNAME/..; echo $PWD)
-INSTALLDIR=$(cd $CLUSTERDIR/../..; echo $PWD)
-$INSTALLDIR/bin/${HELPER_COMMAND} get_cluster_state -quiet
+CLUSTERDIR=$(cd "$DIRNAME/.."; echo $PWD)
+INSTALLDIR=$(cd "$CLUSTERDIR/../.."; echo $PWD)
+"$INSTALLDIR/bin/${HELPER_COMMAND}" get_cluster_state -quiet
 if [ $? -ne 1 ]; then
-  $INSTALLDIR/bin/${HELPER_COMMAND} shutdown_cluster_all
+  "$INSTALLDIR/bin/${HELPER_COMMAND}" shutdown_cluster_all
   first=1
   tries=0
   echo -n "INFO: Waiting up to 60s for cluster to shutdown"
@@ -91,3 +91,4 @@
   fi
 fi
 rm /tmp/$$_pids
+
diff --git 
a/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/SampleLocalClusterIT.java
 
b/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/SampleLocalClusterIT.java
index 61128c5..496d4cf 100644
--- 
a/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/SampleLocalClusterIT.java
+++ 
b/asterixdb/asterix-server/src/test/java/org/apache/asterix/server/test/SampleLocalClusterIT.java
@@ -52,7 +52,7 @@
 
 // Directory where the NCs create and store all data, as configured by
 // src/test/resources/NCServiceExecutionIT/cc.conf.
-private static final String OUTPUT_DIR = FileUtil.joinPath(TARGET_DIR, 
"sample-loca

Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5278/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Handle Spaces in Path For Sample Cluster

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Handle Spaces in Path For Sample Cluster
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/413/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1747
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id4b17698762e3cb2959152e6f957dad175d0f99b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


[jira] [Resolved] (ASTERIXDB-1832) Cluster scripts shouldn't require JDK

2017-05-12 Thread Michael Blow (JIRA)

 [ 
https://issues.apache.org/jira/browse/ASTERIXDB-1832?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Blow resolved ASTERIXDB-1832.
-
Resolution: Fixed

> Cluster scripts shouldn't require JDK
> -
>
> Key: ASTERIXDB-1832
> URL: https://issues.apache.org/jira/browse/ASTERIXDB-1832
> Project: Apache AsterixDB
>  Issue Type: Bug
>  Components: AsterixDB, Documentation, Monitoring/Admin, Other
> Environment: Any/all!  (Mac, Linux, Windows)
>Reporter: Michael J. Carey
>Assignee: Michael Blow
>
> The sample cluster scripts require JDK (not just JRE) currently- this should 
> be fixed- filing an issue for this per MDB's request.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2: BAD-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1156/ : FAILURE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1156/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2790/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/411/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2788/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2789/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/335/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3799/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/595/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/412/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5277/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Yingyi Bu (Code Review)
Yingyi Bu has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

BTW, add a section "Conditional Functions" in 
https://ci.apache.org/projects/asterixdb/sqlpp/builtins.html as well as docs 
for the three functions?  (It could be another change.)

The source is asterixdb/asterix-doc/src/main/markdown/builtins.

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Yingyi Bu (Code Review)
Yingyi Bu has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

(3 comments)

https://asterix-gerrit.ics.uci.edu/#/c/1745/2/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
File 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java:

PS2, Line 892: IfMissingOrNullTypeComputer
Their type computers seem to be different?


https://asterix-gerrit.ics.uci.edu/#/c/1745/2/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/IfMissingOrNullTypeComputer.java
File 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/IfMissingOrNullTypeComputer.java:

PS2, Line 39: 
1. if a leading argument is null/missing, it should look at consequent 
arguments;

2. if an argument is not optional, it doesn't need to look at consequent 
arguments.

Here is my trial for the loop:

for (Mutable argRef : fce.getArguments()) {
 ILogicalExpression arg = argRef.getValue();
 IAType argType = (IAType) env.getType(arg);
 ATypeTag argTypeTag = argType.getTypeTag();
 
 // null or missing
 if (argTypeTag == ATypeTag.NULL || argTypeTag == ATypeTag.MISSING){
 continue;
 } 

  if (resultType == null) {
   resultType = argType;
  }

  //  result type cannot be null or missing
  if (!NonTaggedFormatUtil.isOptional(resultType)){
break;
  }

  AUnionType unionType = (AUnionType) resultType;
  if (unionType.getActualType().equals(argType)){
 resultType = argType; // result type cannot be null or missing.
 break;
  }
  if (!resultType.equals(argType)) {
resultType = BuiltinType.ANY
break;
   }
}


https://asterix-gerrit.ics.uci.edu/#/c/1745/2/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfMissingOrNullDescriptor.java
File 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfMissingOrNullDescriptor.java:

PS2, Line 91: missingBytes
should ifnull(null, null) return null instead of missing?


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: Yes


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/410/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2787/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/334/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3798/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 1:

WARNING: THIS CHANGE CONTAINS CROSS-PRODUCT CHANGES IN:
* asterixdb
* hyracks-fullstack

PLEASE REVIEW CAREFULLY AND LOOK FOR API CHANGES!

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/594/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1746

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..

Added 'source-format' profile to enable code reformatting at build time using 
AsterixCodeFormatProfile

Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
---
A asterixdb/AsterixCodeFormatProfile.xml
M asterixdb/asterix-active/pom.xml
M asterixdb/asterix-algebra/pom.xml
M asterixdb/asterix-app/pom.xml
M asterixdb/asterix-benchmark/pom.xml
M asterixdb/asterix-client-helper/pom.xml
M asterixdb/asterix-common/pom.xml
M asterixdb/asterix-coverage/pom.xml
M asterixdb/asterix-doc/pom.xml
M asterixdb/asterix-docker/pom.xml
M asterixdb/asterix-events/pom.xml
M asterixdb/asterix-examples/pom.xml
M asterixdb/asterix-experiments/pom.xml
M asterixdb/asterix-external-data/pom.xml
M asterixdb/asterix-fuzzyjoin/pom.xml
M asterixdb/asterix-hivecompat/pom.xml
M asterixdb/asterix-installer/pom.xml
M asterixdb/asterix-lang-aql/pom.xml
M asterixdb/asterix-lang-common/pom.xml
M asterixdb/asterix-lang-sqlpp/pom.xml
M 
asterixdb/asterix-maven-plugins/asterix-evaluator-generator-maven-plugin/pom.xml
M asterixdb/asterix-maven-plugins/asterix-grammar-extension-maven-plugin/pom.xml
M 
asterixdb/asterix-maven-plugins/asterix-test-datagenerator-maven-plugin/pom.xml
M asterixdb/asterix-maven-plugins/lexer-generator-maven-plugin/pom.xml
M asterixdb/asterix-maven-plugins/pom.xml
M asterixdb/asterix-maven-plugins/record-manager-generator-maven-plugin/pom.xml
M asterixdb/asterix-metadata/pom.xml
M asterixdb/asterix-om/pom.xml
M asterixdb/asterix-replication/pom.xml
M asterixdb/asterix-runtime/pom.xml
M asterixdb/asterix-server/pom.xml
M asterixdb/asterix-test-framework/pom.xml
M asterixdb/asterix-tools/pom.xml
M asterixdb/asterix-transactions/pom.xml
M asterixdb/asterix-yarn/pom.xml
M asterixdb/pom.xml
A hyracks-fullstack/AsterixCodeFormatProfile.xml
M hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/pom.xml
M hyracks-fullstack/hyracks/hyracks-examples/tpch-example/tpchclient/pom.xml
M hyracks-fullstack/hyracks/hyracks-http/pom.xml
M 
hyracks-fullstack/hyracks/hyracks-maven-plugins/license-automation-plugin/pom.xml
M hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-btree-test/pom.xml
M hyracks-fullstack/hyracks/hyracks-util/pom.xml
M hyracks-fullstack/pom.xml
44 files changed, 797 insertions(+), 20 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/46/1746/1

diff --git a/asterixdb/AsterixCodeFormatProfile.xml 
b/asterixdb/AsterixCodeFormatProfile.xml
new file mode 100644
index 000..4205bb6
--- /dev/null
+++ b/asterixdb/AsterixCodeFormatProfile.xml
@@ -0,0 +1,298 @@
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
diff --git a/asterixdb/asterix-active/pom.xml b/asterixdb/asterix-active/pom.xml
index 950e4d6..4964d2d 100644
--- a/asterixdb/asterix-active/pom.xml
+++ b/asterixdb/asterix-active/pom.xml
@@ -24,6 +24,10 @@
 0.9.2-SNAPSHOT
   
   asterix-active
+  
+${basedir}/..
+false
+  
   
 
   org.apache.asterix
diff --git a/asterixdb/asterix-algebra/pom.xml 
b/asterixdb/asterix-algebra/pom.xml
index e042d97..244a2a4 100644
--- a/asterixdb/asterix-algebra/pom.xml
+++ b/asterixdb/asterix-algebra/pom.xml
@@ -26,7 +26,9 @@
   asterix-algebra
 
   
-
${basedir}/../src/main/appended-resources
+${basedir}/..
+
${root.dir}/src/main/appended-resources
+false
   
 
   
diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml
index 3479ef9..c430600 100644
--- a/asterixdb/asterix-app/pom.xml
+++ b/asterixdb/asterix-app/pom.xml
@@ -33,8 +33,10 @@
 
   
   
+${basedir}/..
 
${basedir}/src/main/appended-resources
 pom.xml,src/main/java,src/main/resources
+false
   
   
 
diff --git a/asterixdb/asterix-benchmark/pom.xml 
b/asterixdb/asterix-benchmark/pom.xml
index 5edb22f..e91218f 100644
--- a/asterixdb/asterix-benchmark/pom.xml
+++ b/asterixdb/asterix-benchmark/pom.xml
@@ -25,6 +25,11 @@
   
   asterix-benchmark
 
+  
+${basedir}/..
+false
+  
+
   
 
   
diff --git a/asterixdb/asterix-client-helper/pom.xml 
b/asterixdb/asterix-client-helper/pom.xml
index 4918c7d..9b3ff01 100644
--- a/asterixdb/asterix-client-helper/pom.xml
+++ b/asterixdb/asterix-client-helper/pom.xml
@@ -24,6 +24,10 @@
 apache-asterixd

Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5276/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Added 'source-format' profile to enable code reformatting at...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Added 'source-format' profile to enable code reformatting at 
build time using AsterixCodeFormatProfile
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/411/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1746
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b5b4b36164c4064d51e078303a5a3f9defc45e0
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Yingyi Bu (Code Review)
Yingyi Bu has submitted this change and it was merged.

Change subject: Update README.
..


Update README.

Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1744
Sonar-Qube: Jenkins 
Reviewed-by: Till Westmann 
Tested-by: Jenkins 
Integration-Tests: Jenkins 
---
M README.md
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Till Westmann: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified

Objections:
  Jenkins: Violations found



diff --git a/README.md b/README.md
index 013f905..4a737cc 100644
--- a/README.md
+++ b/README.md
@@ -27,8 +27,8 @@
 extending JSON with object database ideas
 
 - __Query languages__
-Two expressive and declarative query languages 
([SQL++](http://asterixdb.apache.org/docs/0.9.0/sqlpp/manual.html)
-and [AQL](http://asterixdb.apache.org/docs/0.9.0/aql/manual.html)) that 
support a broad range of queries and analysis
+Two expressive and declarative query languages 
([SQL++](http://asterixdb.apache.org/docs/0.9.1/sqlpp/manual.html)
+and [AQL](http://asterixdb.apache.org/docs/0.9.1/aql/manual.html)) that 
support a broad range of queries and analysis
 over semistructured data
 
 - __Scalability__
@@ -90,6 +90,7 @@
 ## Documentation
 
 * [master](https://ci.apache.org/projects/asterixdb/index.html) |
+  [0.9.1](http://asterixdb.apache.org/docs/0.9.1/index.html) |
   [0.9.0](http://asterixdb.apache.org/docs/0.9.0/index.html)
 
 ## Community support

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2788/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2786/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2: BAD-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1155/ : FAILURE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1155/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/409/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/410/
 (6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2787/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2785/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/408/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2786/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/333/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3797/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/593/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5275/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/409/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Ian Maxon (Code Review)
Hello Jenkins,

I'd like you to reexamine a change.  Please visit

https://asterix-gerrit.ics.uci.edu/1743

to look at the new patch set (#2).

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..

ASTERIXDB-1905: Incorrect filter for post-load sidx

The issue seems to be limited to RTrees. The MBR evaluators
were not accounting for the point MBR optimization as they
were in the compiled load pidx+sidx case

Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
---
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/filters/load-with-secondary-rtree/load-with-secondary-rtree.6.query.aql
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/filters/load-with-secondary-rtree/load-with-secondary-rtree.6.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/filters/load-with-secondary-rtree/load-with-secondary-rtree.2.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/filters/load-with-secondary-rtree/load-with-secondary-rtree.6.ast
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/SecondaryRTreeOperationsHelper.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/base/IDataFormat.java
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/formats/NonTaggedDataFormat.java
7 files changed, 107 insertions(+), 4 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/43/1743/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2: BAD-1

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1154/ : FAILURE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1745/2//COMMIT_MSG
Commit Message:

PS2, Line 1: Parent: 740e6a88 (Rename type check methods into meaningful 
names)
   : Author: Dmitry Lychagin 
   : AuthorDate: 2017-05-12 14:07:10 -0700
   : Commit: Dmitry Lychagin 
   : CommitDate: 2017-05-12 14:07:10 -0700
   : 
   : commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4
   : Author: Dmitry Lychagin 
   : Date:   Fri May 12 13:16:01 2017 -0700
We can this information in. The entire change will be merged as a single commit 
into master and this information is not useful anymore at that point.


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: Yes


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1154/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Till Westmann (Code Review)
Till Westmann has posted comments on this change.

Change subject: Update README.
..


Patch Set 2: Code-Review+2

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/332/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/407/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3796/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2784/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5274/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/592/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2785/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/408/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/331/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3795/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5273/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/591/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: Expose SessionOutput on QueryTranslator

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Expose SessionOutput on QueryTranslator
..


Patch Set 2:

BAD Compatibility Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1153/ : FAILURE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1736
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie784eb021d616c26eb2500af622c8f9b5e092f09
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/407/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: Yingyi Bu 
Gerrit-HasComments: No


Change in asterixdb[master]: Expose SessionOutput on QueryTranslator

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Expose SessionOutput on QueryTranslator
..


Patch Set 2:

BAD Compatibility Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/1153/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1736
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie784eb021d616c26eb2500af622c8f9b5e092f09
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil, IndexUtil, and Metadat...

2017-05-12 Thread Yingyi Bu (Code Review)
Hello Jenkins,

I'd like you to reexamine a change.  Please visit

https://asterix-gerrit.ics.uci.edu/1741

to look at the new patch set (#2).

Change subject: Remove unused methods in DatasetUtil, IndexUtil, and 
MetadataProvider.
..

Remove unused methods in DatasetUtil, IndexUtil, and MetadataProvider.

Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
---
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/DatasetUtil.java
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/IndexUtil.java
3 files changed, 1 insertion(+), 133 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/41/1741/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 


Change in asterixdb[master]: Expose SessionOutput on QueryTranslator

2017-05-12 Thread Till Westmann (Code Review)
Till Westmann has submitted this change and it was merged.

Change subject: Expose SessionOutput on QueryTranslator
..


Expose SessionOutput on QueryTranslator

Change-Id: Ie784eb021d616c26eb2500af622c8f9b5e092f09
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1736
Reviewed-by: abdullah alamoudi 
Integration-Tests: Ian Maxon 
Tested-by: Ian Maxon 
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  abdullah alamoudi: Looks good to me, approved
  Ian Maxon: Verified; Verified



diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
index 5047bc2..b7aae59 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
@@ -229,8 +229,8 @@
 this.executorService = executorService;
 }
 
-public SessionConfig getSessionConfig() {
-return sessionConfig;
+public SessionOutput getSessionOutput() {
+return sessionOutput;
 }
 
 protected List getDeclaredFunctions(List 
statements) {

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1736
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie784eb021d616c26eb2500af622c8f9b5e092f09
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Till Westmann 
Gerrit-Reviewer: abdullah alamoudi 


Change in asterixdb[master]: Expose SessionOutput on QueryTranslator

2017-05-12 Thread Ian Maxon (Code Review)
Ian Maxon has posted comments on this change.

Change subject: Expose SessionOutput on QueryTranslator
..


Patch Set 2: Integration-Tests+1 Verified+1

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1736
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie784eb021d616c26eb2500af622c8f9b5e092f09
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No


Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!

2017-05-12 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has restored this change.

Change subject: PLEASE EDIT to provide a meaningful commit message!
..


Restored

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: restore
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 


Change in asterixdb[master]: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmit...

2017-05-12 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has uploaded a new patch set (#2).

Change subject: commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4 Author: Dmitry 
Lychagin  Date:   Fri May 12 13:16:01 2017 -0700
..

commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4
Author: Dmitry Lychagin 
Date:   Fri May 12 13:16:01 2017 -0700

Added built-in functions: ifmissing(), ifnull(), ifmissingornull()

Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissing/ifmissing.1.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissingornull/ifmissingornull.1.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifnull/ifnull.1.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/types/ifmissing/ifmissing.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/types/ifmissingornull/ifmissingornull.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/types/ifnull/ifnull.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/util/CommonFunctionMapUtil.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/IfMissingOrNullTypeComputer.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfMissingDescriptor.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfMissingOrNullDescriptor.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfNullDescriptor.java
14 files changed, 396 insertions(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/45/1745/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 


Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!

2017-05-12 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has abandoned this change.

Change subject: PLEASE EDIT to provide a meaningful commit message!
..


Abandoned

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1745
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Dmitry Lychagin 


Change in asterixdb[master]: PLEASE EDIT to provide a meaningful commit message!

2017-05-12 Thread Dmitry Lychagin (Code Review)
Dmitry Lychagin has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1745

Change subject: PLEASE EDIT to provide a meaningful commit message!
..

PLEASE EDIT to provide a meaningful commit message!

commit dd5a69be93472bc8a78d10ca7b0f8b8be9412bf4
Author: Dmitry Lychagin 
Date:   Fri May 12 13:16:01 2017 -0700

Added built-in functions: ifmissing(), ifnull(), ifmissingornull()

Change-Id: Id114f6654b9814c5aeca07fffeea04daeb8dca19
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/util/FunctionCollection.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissing/ifmissing.1.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissingornull/ifmissingornull.1.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifnull/ifnull.1.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/types/ifmissing/ifmissing.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/types/ifmissingornull/ifmissingornull.1.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/types/ifnull/ifnull.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/util/CommonFunctionMapUtil.java
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
A 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/IfMissingOrNullTypeComputer.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfMissingDescriptor.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfMissingOrNullDescriptor.java
A 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/IfNullDescriptor.java
14 files changed, 396 insertions(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/45/1745/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 6f24fc5..7431218 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
@@ -158,6 +158,9 @@
 import org.apache.asterix.runtime.evaluators.functions.GramTokensDescriptor;
 import 
org.apache.asterix.runtime.evaluators.functions.HashedGramTokensDescriptor;
 import 
org.apache.asterix.runtime.evaluators.functions.HashedWordTokensDescriptor;
+import org.apache.asterix.runtime.evaluators.functions.IfMissingDescriptor;
+import 
org.apache.asterix.runtime.evaluators.functions.IfMissingOrNullDescriptor;
+import org.apache.asterix.runtime.evaluators.functions.IfNullDescriptor;
 import org.apache.asterix.runtime.evaluators.functions.InjectFailureDescriptor;
 import org.apache.asterix.runtime.evaluators.functions.IsArrayDescriptor;
 import org.apache.asterix.runtime.evaluators.functions.IsBooleanDescriptor;
@@ -428,6 +431,9 @@
 temp.add(IsUnknownDescriptor.FACTORY);
 temp.add(IsSystemNullDescriptor.FACTORY);
 temp.add(CheckUnknownDescriptor.FACTORY);
+temp.add(IfMissingDescriptor.FACTORY);
+temp.add(IfNullDescriptor.FACTORY);
+temp.add(IfMissingOrNullDescriptor.FACTORY);
 
 // uuid generators (zero independent functions)
 temp.add(CreateUUIDDescriptor.FACTORY);
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissing/ifmissing.1.query.sqlpp
 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissing/ifmissing.1.query.sqlpp
new file mode 100644
index 000..db99c32
--- /dev/null
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/types/ifmissing/ifmissing.1.query.sqlpp
@@ -0,0 +1,28 @@
+/*
+ * 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

Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/330/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3794/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/406/ 
(6/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Remove unused methods in DatasetUtil and IndexUtil.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Remove unused methods in DatasetUtil and IndexUtil.
..


Patch Set 1: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2783/ 
: SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1741
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60352d052f684a65fd823a1c7aa33b794bd7f476
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5272/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: ASTERIXDB-1905: Incorrect filter for post-load sidx

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..


Patch Set 1:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/2784/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea158ad4c29ad4421020a28a72e68637bc538560
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Jianfeng Jia 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/590/ (2/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/406/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Update README.

2017-05-12 Thread Yingyi Bu (Code Review)
Yingyi Bu has uploaded a new patch set (#2).

Change subject: Update README.
..

Update README.

Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
---
M README.md
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/44/1744/2
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 


Change in asterixdb[master]: Update README.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Update README.
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/405/
 (1/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1744
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5c7434faa96875554316741d7caffbfe52c0b1
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/329/ (5/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/3793/ (4/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: Add Result Location API.

2017-05-12 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: Add Result Location API.
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/5271/ (3/6)

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1742
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I3c888bb11f90d119b01acc80576e127b84789ed8
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wail Alkowaileet 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


  1   2   >