Change in asterixdb[master]: Don't Start Stop Hyracks Cluster When -DskipTests

2016-06-16 Thread Michael Blow (Code Review) 
Michael Blow has submitted this change and it was merged.

Change subject: Don't Start Stop Hyracks Cluster When -DskipTests
..


Don't Start Stop Hyracks Cluster When -DskipTests

Wrap starting / stopping Hyracks cluster in profile which is disabled
when 'skipTests' is set.

Change-Id: Ifdde6b78fe1b0e9ac76b91a32bff279a1814db51
Reviewed-on: https://asterix-gerrit.ics.uci.edu/931
Reviewed-by: Jenkins 
Tested-by: Jenkins 
Reviewed-by: abdullah alamoudi 
---
M hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
M hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
2 files changed, 161 insertions(+), 133 deletions(-)

Approvals:
  abdullah alamoudi: Looks good to me, approved
  Jenkins: Looks good to me, but someone else must approve; Verified



diff --git 
a/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml 
b/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
index b69b280..63d01ad 100644
--- a/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
+++ b/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
@@ -81,69 +81,6 @@
   
 
   
-org.apache.hyracks
-hyracks-virtualcluster-maven-plugin
-0.2.18-SNAPSHOT
-
-  
${basedir}/target/hyracks-shutdown-test-${project.version}-binary-assembly
-  ${jvm.extraargs}
-
-
-  
-hyracks-cc-start
-pre-integration-test
-
-  start-cc
-
-  
-  
-hyracks-nc1-start
-pre-integration-test
-
-  start-nc
-
-
-  NC1
-  127.0.0.1
-  localhost
-
-  
-  
-hyracks-nc2-start
-pre-integration-test
-
-  start-nc
-
-
-  NC2
-  127.0.0.1
-  localhost
-
-  
-  
-stop-services
-post-integration-test
-
-  stop-services
-
-  
-
-  
-  
-org.apache.maven.plugins
-maven-failsafe-plugin
-2.8.1
-
-  
-it
-integration-test
-
-  integration-test
-
-  
-
-  
-  
 org.apache.maven.plugins
 maven-deploy-plugin
 
@@ -152,6 +89,83 @@
   
 
   
+  
+
+  run-tests
+  
+
+  !skipTests
+
+  
+  
+
+  
+org.apache.hyracks
+hyracks-virtualcluster-maven-plugin
+0.2.18-SNAPSHOT
+
+  
${basedir}/target/hyracks-shutdown-test-${project.version}-binary-assembly
+  ${jvm.extraargs}
+
+
+  
+hyracks-cc-start
+pre-integration-test
+
+  start-cc
+
+  
+  
+hyracks-nc1-start
+pre-integration-test
+
+  start-nc
+
+
+  NC1
+  127.0.0.1
+  localhost
+
+  
+  
+hyracks-nc2-start
+pre-integration-test
+
+  start-nc
+
+
+  NC2
+  127.0.0.1
+  localhost
+
+  
+  
+stop-services
+post-integration-test
+
+  stop-services
+
+  
+
+  
+  
+org.apache.maven.plugins
+maven-failsafe-plugin
+2.8.1
+
+  
+it
+integration-test
+
+  integration-test
+
+  
+
+  
+
+  
+
+  
   
 
   junit
diff --git 
a/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml 
b/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
index f9dc917..babc01e 100644
--- a/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
+++ b/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
@@ -79,78 +79,92 @@
   
 
   
-  
-org.apache.hyracks
-hyracks-virtualcluster-maven-plugin
-0.2.18-SNAPSHOT
-
-  
${basedir}/target/textserver-${project.version}-binary-assembly
-  ${jvm.extraargs}
-
-
-  
-hyracks

Change in asterixdb[master]: Don't Start Stop Hyracks Cluster When -DskipTests

2016-06-16 Thread abdullah alamoudi (Code Review) 
abdullah alamoudi has posted comments on this change.

Change subject: Don't Start Stop Hyracks Cluster When -DskipTests
..


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifdde6b78fe1b0e9ac76b91a32bff279a1814db51
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow 
Gerrit-Reviewer: Ian Maxon 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Don't Start Stop Hyracks Cluster When -DskipTests

2016-06-16 Thread Jenkins (Code Review) 
Jenkins has posted comments on this change.

Change subject: Don't Start Stop Hyracks Cluster When -DskipTests
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/1662/

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

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

Change in asterixdb[master]: Don't Start Stop Hyracks Cluster When -DskipTests

2016-06-16 Thread Michael Blow (Code Review) 
Michael Blow has uploaded a new change for review.

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

Change subject: Don't Start Stop Hyracks Cluster When -DskipTests
..

Don't Start Stop Hyracks Cluster When -DskipTests

Wrap starting / stopping Hyracks cluster in profile which is disabled
when 'skipTests' is set.

Change-Id: Ifdde6b78fe1b0e9ac76b91a32bff279a1814db51
---
M hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
M hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
2 files changed, 161 insertions(+), 133 deletions(-)


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

diff --git 
a/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml 
b/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
index b69b280..63d01ad 100644
--- a/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
+++ b/hyracks-fullstack/hyracks/hyracks-examples/hyracks-shutdown-test/pom.xml
@@ -81,69 +81,6 @@
   
 
   
-org.apache.hyracks
-hyracks-virtualcluster-maven-plugin
-0.2.18-SNAPSHOT
-
-  
${basedir}/target/hyracks-shutdown-test-${project.version}-binary-assembly
-  ${jvm.extraargs}
-
-
-  
-hyracks-cc-start
-pre-integration-test
-
-  start-cc
-
-  
-  
-hyracks-nc1-start
-pre-integration-test
-
-  start-nc
-
-
-  NC1
-  127.0.0.1
-  localhost
-
-  
-  
-hyracks-nc2-start
-pre-integration-test
-
-  start-nc
-
-
-  NC2
-  127.0.0.1
-  localhost
-
-  
-  
-stop-services
-post-integration-test
-
-  stop-services
-
-  
-
-  
-  
-org.apache.maven.plugins
-maven-failsafe-plugin
-2.8.1
-
-  
-it
-integration-test
-
-  integration-test
-
-  
-
-  
-  
 org.apache.maven.plugins
 maven-deploy-plugin
 
@@ -152,6 +89,83 @@
   
 
   
+  
+
+  run-tests
+  
+
+  !skipTests
+
+  
+  
+
+  
+org.apache.hyracks
+hyracks-virtualcluster-maven-plugin
+0.2.18-SNAPSHOT
+
+  
${basedir}/target/hyracks-shutdown-test-${project.version}-binary-assembly
+  ${jvm.extraargs}
+
+
+  
+hyracks-cc-start
+pre-integration-test
+
+  start-cc
+
+  
+  
+hyracks-nc1-start
+pre-integration-test
+
+  start-nc
+
+
+  NC1
+  127.0.0.1
+  localhost
+
+  
+  
+hyracks-nc2-start
+pre-integration-test
+
+  start-nc
+
+
+  NC2
+  127.0.0.1
+  localhost
+
+  
+  
+stop-services
+post-integration-test
+
+  stop-services
+
+  
+
+  
+  
+org.apache.maven.plugins
+maven-failsafe-plugin
+2.8.1
+
+  
+it
+integration-test
+
+  integration-test
+
+  
+
+  
+
+  
+
+  
   
 
   junit
diff --git 
a/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml 
b/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
index f9dc917..babc01e 100644
--- a/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
+++ b/hyracks-fullstack/hyracks/hyracks-examples/text-example/textserver/pom.xml
@@ -79,78 +79,92 @@
   
 
   
-  
-org.apache.hyracks
-hyracks-virtualcluster-maven-plugin
-0.2.18-SNAPSHOT
-
-  
${basedir}/target/textserver-${project.version}-binary-assembly
-  ${jvm.extraargs}
-
-
-  
-hyracks-cc-start
-pre-integration-test
-
-  start-cc
-
-  
-  
-hyrac