Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-12-01 Thread Murtadha Hubail (Code Review)
Murtadha Hubail has abandoned this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Abandoned

With be reimplemented using new storage API

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1: -Integration-Tests

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1: Integration-Tests-1

Integration Tests Timed Out

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4645/ 
: ABORTED

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Anon. E. Moose (Code Review)
Anon. E. Moose #1000171 has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1: Contrib+2

Analytics Compatibility Tests Successful Build 2982
: SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1: Contrib+2

BAD Compatibility Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2162/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2496/ (10/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2241/ (4/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1614/ 
(2/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1626/ 
(3/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5668/ (7/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Murtadha Hubail (Code Review)
Murtadha Hubail has uploaded a new change for review.

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

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..

[NO ISSUE][REPL] Add MetadataReplication Runtime Tests

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Run Sql++ runtime tests with metadata rep enabled.
- Skip temporary datasets replication.
- Properly close replication channel on shutdown.
- Wait for remote primary replica on shutdown.

Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
---
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithMetadataRepTest.java
M 
asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/management/ReplicationChannel.java
M 
asterixdb/asterix-replication/src/main/java/org/apache/asterix/replication/management/ReplicationManager.java
4 files changed, 95 insertions(+), 12 deletions(-)


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

diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
index 78bed6a..cd49600 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
@@ -39,8 +39,8 @@
 import org.apache.commons.lang.SystemUtils;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.hyracks.api.io.IODeviceHandle;
-import org.apache.hyracks.util.ThreadDumpUtil;
 import org.apache.hyracks.control.nc.NodeControllerService;
+import org.apache.hyracks.util.ThreadDumpUtil;
 
 /**
  * Utils for running SQL++ or AQL runtime tests.
@@ -58,8 +58,14 @@
 
 private static ExternalUDFLibrarian librarian;
 private static final int repeat = Integer.getInteger("test.repeat", 1);
+private static boolean checkStorageDistribution = true;
 
 public static void setUp(String configFile, TestExecutor executor) throws 
Exception {
+setUp(configFile, executor, checkStorageDistribution);
+}
+
+public static void setUp(String configFile, TestExecutor executor, boolean 
checkStorageDistribution)
+throws Exception {
 testExecutor = executor;
 File outdir = new File(PATH_ACTUAL);
 outdir.mkdirs();
@@ -70,6 +76,7 @@
 if (repeat != 1) {
 System.out.println("FYI: each test will be run " + repeat + " 
times.");
 }
+LangExecutionUtil.checkStorageDistribution = checkStorageDistribution;
 }
 
 public static void tearDown() throws Exception {
@@ -125,7 +132,9 @@
 testExecutor.executeTest(PATH_ACTUAL, tcCtx, null, false, 
ExecutionTestUtil.FailedGroup);
 
 try {
-checkStorageFiles();
+if (checkStorageDistribution) {
+checkStorageFiles();
+}
 } finally {
 testExecutor.cleanup(tcCtx.toString(), badTestCases);
 }
diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithMetadataRepTest.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithMetadataRepTest.java
new file mode 100644
index 000..65fdf83
--- /dev/null
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithMetadataRepTest.java
@@ -0,0 +1,67 @@
+/*
+ * 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.
+ */
+package org.apache.asterix.test.runtime;
+
+import java.util.Collection;
+
+import org.apache.asterix.common.config.ClusterProperties;
+import org.apache.asterix.test.common.TestExecutor;
+import org.apache.asterix.testframework.context.TestCaseContext;
+import org.junit.AfterClass;
+import org.junit.BeforeClass;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import 

Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7166/ (8/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1948/ 
(6/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/812/
 (5/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb[master]: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests

2017-11-17 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][REPL] Add MetadataReplication Runtime Tests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1633/
 (1/11)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f5b5b35063cc5fe466e699d36a47b01c4a97a11
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No