You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Yingyi Bu (Code Review)" <do...@asterixdb.incubator.apache.org> on 2017/02/17 20:30:47 UTC

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Yingyi Bu has uploaded a new change for review.

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

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................

Use rsync instead of wget to distribute the benchmark source.

Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
---
M asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
M asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
2 files changed, 5 insertions(+), 6 deletions(-)


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

diff --git a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
index a0aa8d3..6aca279 100644
--- a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
+++ b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
@@ -32,11 +32,10 @@
     path: "{{ generatorzip }}"
     state: absent
 
-- name: Download TPC-H data generator
-  get_url:
-    url: "{{ tpch_url }}"
+- name: Upload TPC-H data generator
+  synchronize:
+    src: "{{ tpch_path }}"
     dest: "{{ home_dir }}"
-    force: yes
 
 - name: Unzip data generator
   shell: unzip "{{ generatorzip }}"
diff --git a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
index 11ba53c..5a0a3ea 100644
--- a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
+++ b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
@@ -41,5 +41,5 @@
 # The local data generation command.
 localgencmd: "gen.sh"
 
-# The url for the TPC-H benchmark.
-tpch_url: <to be filed>
\ No newline at end of file
+# The path for the TPC-H benchmark source zip.
+tpch_path: <an absolute path to be filed>
\ No newline at end of file

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has submitted this change and it was merged.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Use rsync instead of wget to distribute the benchmark source.

Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1511
Reviewed-by: Ian Maxon <im...@apache.org>
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Integration-Tests: Ian Maxon <im...@apache.org>
---
M asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
M asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
2 files changed, 5 insertions(+), 6 deletions(-)

Approvals:
  Ian Maxon: Looks good to me, approved; Verified
  Jenkins: Verified; No violations found



diff --git a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
index a0aa8d3..6aca279 100644
--- a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
+++ b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/compile.yml
@@ -32,11 +32,10 @@
     path: "{{ generatorzip }}"
     state: absent
 
-- name: Download TPC-H data generator
-  get_url:
-    url: "{{ tpch_url }}"
+- name: Upload TPC-H data generator
+  synchronize:
+    src: "{{ tpch_path }}"
     dest: "{{ home_dir }}"
-    force: yes
 
 - name: Unzip data generator
   shell: unzip "{{ generatorzip }}"
diff --git a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
index 11ba53c..5a0a3ea 100644
--- a/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
+++ b/asterixdb/asterix-benchmark/src/main/resources/benchmarks/tpch/gen/settings.yml
@@ -41,5 +41,5 @@
 # The local data generation command.
 localgencmd: "gen.sh"
 
-# The url for the TPC-H benchmark.
-tpch_url: <to be filed>
\ No newline at end of file
+# The path for the TPC-H benchmark source zip.
+tpch_path: <an absolute path to be filed>
\ No newline at end of file

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1:

Integration Tests Successful

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Ian Maxon (Code Review)" <do...@asterixdb.incubator.apache.org>.
Ian Maxon has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1: Integration-Tests+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No

Change in asterixdb[master]: Use rsync instead of wget to distribute the benchmark source.

Posted by "Jenkins (Code Review)" <do...@asterixdb.incubator.apache.org>.
Jenkins has posted comments on this change.

Change subject: Use rsync instead of wget to distribute the benchmark source.
......................................................................


Patch Set 1: BAD-1

BAD Compatibility Tests Failed

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I58d2553eb7866d6cc947f9a8f0443cb82fc654f6
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-HasComments: No