Change in asterixdb-bad[master]: [NO ISSUE][BAD] BAD clean-ups

2018-11-12 Thread Xikui Wang (Code Review)
Xikui Wang has abandoned this change.

Change subject: [NO ISSUE][BAD] BAD clean-ups
..


Abandoned

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: I76c5d201a95f71a8f9bfd2461620145c0c27a08d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: James Fang 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: Till Westmann 


Change in asterixdb-bad[master]: [NO ISSUE][BAD] BAD clean-ups

2018-11-06 Thread Xikui Wang (Code Review)
Hello Jenkins,

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

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

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

Change subject: [NO ISSUE][BAD] BAD clean-ups
..

[NO ISSUE][BAD] BAD clean-ups

Details:
1. Cleaned up dependencies which are not used.
2. Changed logging to use log4j.
3. Refactored BADExecutionTest to be consistent with master.
4. Cleaned up test cases to make sure every test case is independent.
5. Unused file removed.

Change-Id: I76c5d201a95f71a8f9bfd2461620145c0c27a08d
---
M asterix-bad/pom.xml
M asterix-bad/src/main/java/org/apache/asterix/bad/BADJobService.java
M 
asterix-bad/src/main/java/org/apache/asterix/bad/metadata/DeployedJobSpecEventListener.java
M 
asterix-bad/src/main/java/org/apache/asterix/bad/recovery/BADGlobalRecoveryManager.java
M 
asterix-bad/src/main/java/org/apache/asterix/bad/runtime/NotifyBrokerRuntime.java
D asterix-bad/src/main/resources/asterix-build-configuration.xml
M asterix-bad/src/main/resources/cc.conf
M asterix-bad/src/test/java/org/apache/asterix/bad/test/BADExecutionTest.java
M asterix-bad/src/test/java/org/apache/asterix/bad/test/BADOptimizerTest.java
M asterix-bad/src/test/java/org/apache/asterix/bad/test/BADRecoveryTest.java
D asterix-bad/src/test/resources/conf/cluster.xml
D asterix-bad/src/test/resources/conf/hyracks-deployment.properties
A asterix-bad/src/test/resources/conf/log4j2-bad.xml
D asterix-bad/src/test/resources/conf/test.properties
A asterix-bad/src/test/resources/runtimets/only_bad.xml
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_function/drop_function.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_function_dataverse/drop_function_dataverse.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_index/drop_index.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_results/drop_results.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_subscriptions/drop_subscriptions.1.ddl.sqlpp
A 
asterix-bad/src/test/resources/runtimets/queries/procedure/create_procedure_check_metadata/create_procedure_check_metadata.3.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/delete_procedure_drop_dataset/delete_procedure_drop_dataset.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/delete_procedure_drop_function/delete_procedure_drop_function.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/delete_procedure_drop_index/delete_procedure_drop_index.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/insert_procedure_drop_dataset/insert_procedure_drop_dataset.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/insert_procedure_drop_dataverse/insert_procedure_drop_dataverse.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/query_procedure_drop_dataset/query_procedure_drop_dataset.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/query_procedure_drop_function/query_procedure_drop_function.1.ddl.sqlpp
R asterix-bad/src/test/resources/runtimets/testsuite_bad.xml
29 files changed, 251 insertions(+), 405 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I76c5d201a95f71a8f9bfd2461620145c0c27a08d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 


Change in asterixdb-bad[master]: [NO ISSUE][BAD] BAD clean-ups

2018-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][BAD] BAD clean-ups
..


Patch Set 2:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/359/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I76c5d201a95f71a8f9bfd2461620145c0c27a08d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb-bad[master]: [NO ISSUE][BAD] BAD clean-ups

2018-11-06 Thread Jenkins (Code Review)
Jenkins has posted comments on this change.

Change subject: [NO ISSUE][BAD] BAD clean-ups
..


Patch Set 1:

Build Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-gerrit/358/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I76c5d201a95f71a8f9bfd2461620145c0c27a08d
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb-bad
Gerrit-Branch: master
Gerrit-Owner: Xikui Wang 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No


Change in asterixdb-bad[master]: [NO ISSUE][BAD] BAD clean-ups

2018-11-06 Thread Xikui Wang (Code Review)
Xikui Wang has uploaded a new change for review.

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

Change subject: [NO ISSUE][BAD] BAD clean-ups
..

[NO ISSUE][BAD] BAD clean-ups

Details:
1. Cleaned up dependencies which are not used.
2. Changed logging to use log4j.
3. Refactored BADExecutionTest to be consistent with master.
4. Cleaned up test cases to make sure every test case is independent.
5. Unused file removed.

Change-Id: I76c5d201a95f71a8f9bfd2461620145c0c27a08d
---
M asterix-bad/pom.xml
M asterix-bad/src/main/java/org/apache/asterix/bad/BADJobService.java
M 
asterix-bad/src/main/java/org/apache/asterix/bad/metadata/DeployedJobSpecEventListener.java
M 
asterix-bad/src/main/java/org/apache/asterix/bad/recovery/BADGlobalRecoveryManager.java
M 
asterix-bad/src/main/java/org/apache/asterix/bad/runtime/NotifyBrokerRuntime.java
D asterix-bad/src/main/resources/asterix-build-configuration.xml
M asterix-bad/src/main/resources/cc.conf
M asterix-bad/src/test/java/org/apache/asterix/bad/test/BADExecutionTest.java
M asterix-bad/src/test/java/org/apache/asterix/bad/test/BADOptimizerTest.java
M asterix-bad/src/test/java/org/apache/asterix/bad/test/BADRecoveryTest.java
D asterix-bad/src/test/resources/conf/cluster.xml
D asterix-bad/src/test/resources/conf/hyracks-deployment.properties
A asterix-bad/src/test/resources/conf/log4j2-bad.xml
D asterix-bad/src/test/resources/conf/test.properties
A asterix-bad/src/test/resources/runtimets/only_bad.xml
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_function/drop_function.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_function_dataverse/drop_function_dataverse.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_index/drop_index.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_results/drop_results.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/channel/drop_subscriptions/drop_subscriptions.1.ddl.sqlpp
A 
asterix-bad/src/test/resources/runtimets/queries/procedure/create_procedure_check_metadata/create_procedure_check_metadata.3.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/delete_procedure_drop_dataset/delete_procedure_drop_dataset.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/delete_procedure_drop_function/delete_procedure_drop_function.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/delete_procedure_drop_index/delete_procedure_drop_index.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/insert_procedure_drop_dataset/insert_procedure_drop_dataset.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/insert_procedure_drop_dataverse/insert_procedure_drop_dataverse.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/query_procedure_drop_dataset/query_procedure_drop_dataset.1.ddl.sqlpp
M 
asterix-bad/src/test/resources/runtimets/queries/procedure/query_procedure_drop_function/query_procedure_drop_function.1.ddl.sqlpp
R asterix-bad/src/test/resources/runtimets/testsuite_bad.xml
29 files changed, 251 insertions(+), 405 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb-bad 
refs/changes/22/3022/1

diff --git a/asterix-bad/pom.xml b/asterix-bad/pom.xml
index d6c0602..6396b36 100644
--- a/asterix-bad/pom.xml
+++ b/asterix-bad/pom.xml
@@ -28,7 +28,7 @@
 0.9.5-SNAPSHOT
 0.3.5-SNAPSHOT
 true
-
${root.dir}/../../asterix-app/src/test/resources/log4j2-test.xml
+
${root.dir}/src/test/resources/conf/log4j2-bad.xml
 
 
 
@@ -153,25 +153,16 @@
   2.10
   
 
-  
org.apache.hadoop:hadoop-minicluster
   
org.apache.asterix:asterix-fuzzyjoin
+  
org.apache.hyracks:hyracks-storage-am-lsm-btree-test
+  
org.apache.asterix:asterix-common:test-jar
 
-
-  
org.apache.asterix:asterix-common
-
   
 
   
 
   
   
-
-  org.apache.hadoop
-  hadoop-minicluster
-  ${hadoop.version}
-  jar
-  test
-
 
   org.apache.asterix
   asterix-om
@@ -225,13 +216,6 @@
   test
 
 
-  org.apache.asterix
-  asterix-common
-  ${asterix.version}
-  test-jar
-  test
-
-
   org.apache.hyracks
   algebricks-common
 
@@ -248,10 +232,6 @@
   org.apache.asterix
   asterix-lang-common
   ${asterix.version}
-
-
-  log4j
-  log4j
 
 
   org.apache.commons
@@ -307,7 +287,22 @@
 
   commons-io
   commons-io
-  2.6
+
+
+  org.apache.hyracks
+  hyracks-api
+
+
+  org.apache.logging.log4j
+  log4j-api
+
+
+
+  org.apache.hyracks
+  hyracks-storage-am-lsm-btree-test
+  ${hyracks.version}
+  test-jar
+  test