[kudu-CR] [consensus] KUDU-1613: Fix replica eviction failure for WRONG SERVER UUID

2016-12-03 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5111 to look at the new patch set (#5). Change subject: [consensus] KUDU-1613: Fix replica eviction failure for WRONG_SERVER_UUID

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Add integration tests for duplicate keys .. Add integration tests for duplicate keys This splits the "crashy nodes" and "churny elections" test

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread .. KUDU-798 (part 3) Replica transactions must start/abort

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add integration tests for duplicate keys .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5349 To unsubscribe, visit

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add integration tests for duplicate keys .. Patch Set 8: (1 comment)

[kudu-CR] ts recovery-itest: reduce test flakiness

2016-12-03 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5351 to review the following change. Change subject: ts_recovery-itest: reduce test flakiness ..

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add integration tests for duplicate keys .. Patch Set 9: (1 comment)

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#9). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add integration tests for duplicate keys .. Patch Set 8: (1 comment)

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread .. Patch Set 21: Code-Review+2 (1 comment)

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add integration tests for duplicate keys .. Patch Set 8: (2 comments)

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread .. Patch Set 21: Safe time advancement

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#21). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#20). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#8). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add integration tests for duplicate keys .. Patch Set 7: (10 comments)

[kudu-CR] [consensus] KUDU-1613: Fix replica eviction failure for WRONG SERVER UUID

2016-12-03 Thread Dinesh Bhat (Code Review)
Hello David Ribeiro Alves, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5111 to look at the new patch set (#4). Change subject: [consensus] KUDU-1613: Fix replica eviction failure for WRONG_SERVER_UUID

[kudu-CR] [consensus] KUDU-1613: Fix replica eviction failure for WRONG SERVER UUID

2016-12-03 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [consensus] KUDU-1613: Fix replica eviction failure for WRONG_SERVER_UUID .. Patch Set 3: (4 comments)

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#19). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#18). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#7). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#17). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#16). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#6). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#5). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#4). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#3). Change subject: Add integration tests for duplicate keys .. Add integration

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#15). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] Add raft integration tests with duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5349 to look at the new patch set (#2). Change subject: Add raft integration tests with duplicate keys .. Add raft

[kudu-CR] KUDU-1753 [delete table-test] deleted-while-scanned test

2016-12-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5345 to look at the new patch set (#5). Change subject: KUDU-1753 [delete_table-test] deleted-while-scanned test ..

[kudu-CR] KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

2016-12-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5294 to look at the new patch set (#14). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] Add integration tests for duplicate keys

2016-12-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5349 Change subject: Add integration tests for duplicate keys .. Add integration tests for duplicate keys This splits the "crashy

[kudu-CR] [i-tests] TestWorkload::set num tablets() accepts 1

2016-12-03 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [i-tests] TestWorkload::set_num_tablets() accepts 1 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5347 To unsubscribe, visit

[kudu-CR] java tests: Clean up and document binDir search

2016-12-03 Thread Mike Percy (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5348 to review the following change. Change subject: java tests: Clean up and document binDir search .. java tests: Clean

[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-03 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: java tests: Also search for build bindir from cwd .. Abandoned OK, I'll abandon since we have a workaround. I'll post a separate CR to document this inline so we don't

[kudu-CR] KUDU-1753 [tablet service] continue scan request on deleted tablet

2016-12-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5346 to look at the new patch set (#3). Change subject: KUDU-1753 [tablet_service] continue scan request on deleted tablet

[kudu-CR] KUDU-1753 [tablet service] continue scan request on deleted tablet

2016-12-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1753 [tablet_service] continue scan request on deleted tablet .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1753 [delete table-test] deleted-while-scanned test

2016-12-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5345 to look at the new patch set (#4). Change subject: KUDU-1753 [delete_table-test] deleted-while-scanned test ..