[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Integrate the result tracker with writes .. Integrate the result tracker with writes This patch integrates the result tracker with write transacti

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 28: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3449 To unsubscribe, visit http:/

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 28: Code-Review+2 rebased this due to a problem with the mm patch. keeping the +2 -- To view, visit

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 28: Build Started http://104.196.14.100/job/kudu-gerrit/2491/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 27: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3449 To unsubscribe, visit http:/

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 26: (2 comments) http://gerrit.cloudera.org:8080/#/c/3449/26/src/kudu/tablet/transactions/write_tran

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#26). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#27). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 27: Build Started http://104.196.14.100/job/kudu-gerrit/2488/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 26: (2 comments) http://gerrit.cloudera.org:8080/#/c/3449/26/src/kudu/tablet/transactions/write_trans

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 26: Build Started http://104.196.14.100/job/kudu-gerrit/2484/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: (1 comment) http://gerrit.cloudera.org:8080/#/c/3449/25/src/kudu/tablet/transactions/transaction

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: (13 comments) http://gerrit.cloudera.org:8080/#/c/3449/25/src/kudu/tablet/transactions/transacti

[kudu-CR] Integrate the result tracker with writes

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: (14 comments) Just nits. http://gerrit.cloudera.org:8080/#/c/3449/25/src/kudu/tablet/transaction

[kudu-CR] Integrate the result tracker with writes

2016-07-13 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 25: Build Started http://104.196.14.100/job/kudu-gerrit/2381/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 24: Verified+1 bunch of unrelated failures with no logs. guess Im ddos'ing the dist test system --

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#24). Change subject: Integrate the result tracker with writes .. Integrate the result tracker with writes This patch integrates the result tracker with write transactions, inclu

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 24: Build Started http://104.196.14.100/job/kudu-gerrit/2371/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 23: Verified+1 unrelated flakyness, addressed that on a previous patch -- To view, visit http://ger

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 23: Build Started http://104.196.14.100/job/kudu-gerrit/2358/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 22: seems like I missed enabling exactly once for remote bootstrap tests. -- To view, visit http://

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 22: Build Started http://104.196.14.100/job/kudu-gerrit/2347/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 21: Build Started http://104.196.14.100/job/kudu-gerrit/2341/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#21). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 17: (18 comments) http://gerrit.cloudera.org:8080/#/c/3449/17/src/kudu/client/batcher.cc File src/ku

[kudu-CR] Integrate the result tracker with writes

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 20: Build Started http://104.196.14.100/job/kudu-gerrit/2289/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 19: Build Started http://104.196.14.100/job/kudu-gerrit/2286/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 18: Build Started http://104.196.14.100/job/kudu-gerrit/2281/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-08 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#18). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 17: (1 comment) http://gerrit.cloudera.org:8080/#/c/3449/17/src/kudu/rpc/result_tracker.cc File src/

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 17: (18 comments) http://gerrit.cloudera.org:8080/#/c/3449/17/src/kudu/client/batcher.cc File src/kudu/clien

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 17: (11 comments) Here a bunch of comments I had pending on PS1 (starting to review again now the latest pa

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 17: Build Started http://104.196.14.100/job/kudu-gerrit/2204/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#17). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 16: Build Started http://104.196.14.100/job/kudu-gerrit/2200/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#16). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 15: Build Started http://104.196.14.100/job/kudu-gerrit/2197/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#15). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 14: Build Started http://104.196.14.100/job/kudu-gerrit/2190/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#14). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#13). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 13: Build Started http://104.196.14.100/job/kudu-gerrit/2176/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/2172/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#12). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2169/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#11). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/2159/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Integrate the result tracker with writes

2016-07-05 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#10). Change subject: Integrate the result tracker with writes .. Integrate the resul

[kudu-CR] Integrate the result tracker with writes

2016-06-30 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 9: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2140/ -- To view, visit http://gerrit.clo

[kudu-CR] Integrate the result tracker with writes

2016-06-30 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/2131/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 8: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2087/ -- To view, visit http://gerrit.clo

[kudu-CR] Integrate the result tracker with writes

2016-06-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 8: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2081/ -- To view, visit http://gerrit.clo

[kudu-CR] Integrate the result tracker with writes

2016-06-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/3449 to look at the new patch set (#8). Change subject: Integrate the result tracker with writes .. Integrate the result

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 7: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2050/ -- To view, visit http://gerrit.clo

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#7). Change subject: Integrate the result tracker with writes .. Integrate the result

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2042/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/2036/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 5: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/2028/ -- To view, visit http://gerrit.clo

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3449 to look at the new patch set (#5). Change subject: Integrate the result tracker with writes .. Integrate the result

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2020/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-27 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2010/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2003/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: Integrate the result tracker with writes .. Integrate the result tracker with writes This patch integrates the result tracker with write transactions, includ

[kudu-CR] Integrate the result tracker with writes

2016-06-22 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 1: Verified+1 unrelated flake org.kududb.client.TestTimeouts.org.kududb.client.TestTimeouts -- To v

[kudu-CR] Integrate the result tracker with writes

2016-06-22 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 1: this is good to be reviewed, but it's still missing a flag to disable it until we've finished gar

[kudu-CR] Integrate the result tracker with writes

2016-06-22 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Integrate the result tracker with writes .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1929/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] Integrate the result tracker with writes

2016-06-22 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3449 Change subject: Integrate the result tracker with writes .. Integrate the result tracker with writes This patch integrates the res