[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: > Hrm, it sounds like the subsequent patch rewrites/improves a lot of > what's in this one. Why not squash

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: Hrm, it sounds like the subsequent patch rewrites/improves a lot of what's in this one. Why not squash the

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (1 comment) BTW, what would be the best way to update this patch given the fact there are subsequent ones?

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (4 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (18 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (3 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#12). Change subject: Kudu Jepsen Tests - Initial Commit .. Kudu Jepsen Tests -

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#11). Change subject: Kudu Jepsen Tests - Initial Commit .. Kudu Jepsen Tests -

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-29 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#10). Change subject: Kudu Jepsen Tests - Initial Commit .. Kudu Jepsen Tests -

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/5492/9/build-support/jenkins/toolchains.xml File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 6: (2 comments) Thank you for the review! Certainly we should communicate on our expectations and

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5492/6/java/pom.xml File java/pom.xml: Line 308:

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/5492/9/build-support/jenkins/build-and-test.sh File

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-28 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#9). Change subject: Kudu Jepsen Tests - Initial Commit .. Kudu Jepsen Tests -

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-28 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#7). Change subject: Kudu Jepsen Tests - Initial Commit .. Kudu Jepsen Tests -

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-27 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 6: (11 comments) I only skimmed the Jepsen code; mostly I was looking at how this integrates into the rest of the

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5492/4/java/kudu-jepsen/src/main/clojure/jepsen/kudu.clj

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2016-12-22 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#6). Change subject: Kudu Jepsen Tests - Initial Commit .. Kudu Jepsen Tests -