[kudu-CR] WIP: bug in exactly-once during tablet bootstrap

2016-12-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/5417 to look at the new patch set (#2). Change subject: WIP: bug in exactly-once during tablet bootstrap .. WIP: bug in

[kudu-CR] WIP: bug in exactly-once during tablet bootstrap

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: bug in exactly-once during tablet bootstrap .. Patch Set 2: oops sorry ended up rebasing your patch on master -- To view, visit

[kudu-CR] KUDU-798 (part 5) Correct safe time advancement

2016-12-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#30). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798

[kudu-CR] KUDU-798 (part 5) Correct safe time advancement

2016-12-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 5) Correct safe time advancement .. Patch Set 28: (12 comments) http://gerrit.cloudera.org:8080/#/c/5240/27//COMMIT_MSG Commit Message:

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

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 [delete_table-test] deleted-while-scanned test .. Patch Set 1: (10 comments)

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

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves 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] 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-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 a way to get the physical time component of a hybrid timestamp

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#3). Change subject: Add a way to get the physical time component of a hybrid timestamp

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#18). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

[kudu-CR] Allow to build a MonoTime from a MonoDelta

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5325 Change subject: Allow to build a MonoTime from a MonoDelta .. Allow to build a MonoTime from a MonoDelta This allows to build a

[kudu-CR] Allow to build a MonoTime from a MonoDelta

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to build a MonoTime from a MonoDelta .. Patch Set 1: so that we can build a monotime from a timestamp (like the physical part of a timestamp) and

[kudu-CR] Allow to build a MonoTime from a MonoDelta

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: Allow to build a MonoTime from a MonoDelta .. Allow to build a MonoTime from a MonoDelta This allows to build a MonoTime from a timestamp representing time

[kudu-CR] Make linked list-test finish with snapshot scans in the present

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: Make linked_list-test finish with snapshot scans in the present .. Abandoned merged this with another patch -- To view, visit

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#19). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

[kudu-CR] Remove the clock from MvccManager

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5326 Change subject: Remove the clock from MvccManager .. Remove the clock from MvccManager Now that safe time has it's own managing

[kudu-CR] WIP KUDU-1127 Don't hang scanner threads waiting for safe time

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5305 to look at the new patch set (#2). Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#5). Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a

[kudu-CR] KUDU-625 - Make write transactions abort in mvcc, by default

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-625 - Make write transactions abort in mvcc, by default .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/5320/5//COMMIT_MSG Commit

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get a MonoDelta from two Timestamps .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/5304/8/src/kudu/server/hybrid_clock-test.cc

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a MonoDelta from two Timestamps This adds a new method to Clock: MonoDelta

[kudu-CR] KUDU-625 - Make write transactions abort in mvcc, by default

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-625 - Make write transactions abort in mvcc, by default .. KUDU-625 - Make write transactions abort in mvcc, by default This patch addresses

[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] test for timestamp propagation with write ops

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [i-tests] test for timestamp propagation with write ops .. Patch Set 1: (1 comment)

[kudu-CR] [c++ client] propagate timestamp for write operations

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] propagate timestamp for write operations .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5269 To

[kudu-CR] [c++ client] timestamp propagation via scan tokens

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] timestamp propagation via scan tokens .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5220 To unsubscribe,

[kudu-CR] [i-tests] scan token timestamp propagation test

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [i-tests] scan token timestamp propagation test .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5219 To unsubscribe,

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-30 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#8). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] [i-tests] test for timestamp propagation with write ops

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [i-tests] test for timestamp propagation with write ops .. Patch Set 1: Mike: We've been pushing the fix and the test separately for this stuff. I think

[kudu-CR] KUDU-1770 [i-tests] test for timestamp propagation with write ops

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1770 [i-tests] test for timestamp propagation with write ops .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5268

[kudu-CR] KUDU-420 [i-tests] scan token timestamp propagation test

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-420 [i-tests] scan token timestamp propagation test .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5219 To

[kudu-CR] [consensus] KUDU-1718: Fix few bugs around replica eviction failures

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [consensus] KUDU-1718: Fix few bugs around replica eviction failures .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1770 [c++ client] propagate timestamp for write operations

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1770 [c++ client] propagate timestamp for write operations .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5269 To

[kudu-CR] KUDU-1770 [c++ client] propagate timestamp for write operations

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1770 [c++ client] propagate timestamp for write operations .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5269 To

[kudu-CR] KUDU-1770 [i-tests] test for timestamp propagation with write ops

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1770 [i-tests] test for timestamp propagation with write ops .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5268

[kudu-CR] KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

2016-12-01 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 (#5). Change subject: KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

[kudu-CR] WIP KUDU-1127 Don't hang scanner threads waiting for safe time

2016-12-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time .. Patch Set 1: btw I think the solution for that would actually more in the line of making

[kudu-CR] KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

2016-12-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread .. Patch Set 6: (3 comments)

[kudu-CR] KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

2016-12-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread .. Patch Set 3: (8 comments)

[kudu-CR] KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

2016-12-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread .. Patch Set 4: (1 comment)

[kudu-CR] WIP KUDU-1127 Don't hang scanner threads waiting for safe time

2016-12-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time .. Patch Set 1: (1 comment) Yeah this doesn't try to make it not wait at all, just tries

[kudu-CR] KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

2016-12-01 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 (#8). Change subject: KUDU-798 (part 3) Make replica transactions start/abort on the consensus thread

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

2016-12-02 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 (#9). Change subject: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] KUDU-625 - Make write transactions abort in mvcc, by default

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5320 to look at the new patch set (#3). Change subject: KUDU-625 - Make write transactions abort in mvcc, by default ..

[kudu-CR] [flaky tests] Fix "Already present" failures on raft consensus-itest

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: [flaky tests] Fix "Already present" failures on raft_consensus-itest .. [flaky tests] Fix "Already present" failures on raft_consensus-itest In

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5240 to look at the new patch set (#21). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get a MonoDelta from two Timestamps .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/5304/3/src/kudu/server/hybrid_clock-test.cc

[kudu-CR] WIP KUDU-1127 Don't hang scanner threads waiting for safe time

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has restored this change. Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time .. Restored oops abandoned wrong patch -- To view, visit http://gerrit.cloudera.org:8080/5305 To

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#9). Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a

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

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread .. Patch Set 12: no, I'll clean this up and run the

[kudu-CR] Allow to build a MonoTime from a MonoDelta

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: Allow to build a MonoTime from a MonoDelta .. Abandoned abandoning this for a better solution -- To view, visit http://gerrit.cloudera.org:8080/5325 To

[kudu-CR] [flaky tests] Fix "Already present" failures on raft consensus-itest

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5319 to look at the new patch set (#4). Change subject: [flaky tests] Fix "Already present" failures on raft_consensus-itest

[kudu-CR] KUDU-625 - Make write transactions abort in mvcc, by default

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5320 to look at the new patch set (#7). Change subject: KUDU-625 - Make write transactions abort in mvcc, by default

[kudu-CR] raft consensus-itest: add a test for a potential deadlock

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: raft_consensus-itest: add a test for a potential deadlock .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5341 To

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get a MonoDelta from two Timestamps .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5304/4/src/kudu/server/hybrid_clock-test.cc

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#6). Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a

[kudu-CR] raft consensus-itest: add a test for a potential deadlock

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: raft_consensus-itest: add a test for a potential deadlock .. raft_consensus-itest: add a test for a potential deadlock This adds a test case

[kudu-CR] WIP KUDU-1127 Don't hang scanner threads waiting for safe time

2016-12-02 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time .. Abandoned decided to go another way -- To view, visit http://gerrit.cloudera.org:8080/5305 To

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

2016-12-02 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 (#13). Change subject: WIP: KUDU-798 (part 3) Replica transactions must start/abort on the consensus update thread

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#4). Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#7). Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a

[kudu-CR] Allow to get a MonoDelta from two Timestamps

2016-12-02 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5304 to look at the new patch set (#8). Change subject: Allow to get a MonoDelta from two Timestamps .. Allow to get a

[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] 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] 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] 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] 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 (#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 (#6). 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 (#16). 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 (#17). 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 (#18). 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 (#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)
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] 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 9: (1 comment)

[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] 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 snapshot scans to fuzz-itest

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-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/5240 to look at the new patch set (#7). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. Patch Set 24: Code-Review+2 just a rebase keeping todd's +2 -- To view, visit

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-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/5240 to look at the new patch set (#6). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Don't output unobservable rows from the MemRowset .. Patch Set 17: Code-Review+1 (5 comments) just a rebase, keeping todd's +1

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add snapshot scans to fuzz-itest .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4996/12/src/kudu/integration-tests/fuzz-itest.cc File

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-29 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4994 to look at the new patch set (#16). Change subject: Don't output unobservable rows from the MemRowset

[kudu-CR] WIP: Support safe time advancement on replicas in the absense of writes

2016-11-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/5240 to look at the new patch set (#5). Change subject: WIP: Support safe time advancement on replicas in the absense of writes

[kudu-CR] KUDU-237 (part 2) - Add support for REINSERT in delta files

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-237 (part 2) - Add support for REINSERT in delta files .. KUDU-237 (part 2) - Add support for REINSERT in delta files This patch goes the

[kudu-CR] Add snapshot scans to fuzz-itest

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Add snapshot scans to fuzz-itest .. Add snapshot scans to fuzz-itest This adds a new operation to fuzz-itest: snapshot scans at a timestamp.

[kudu-CR] Don't output unobservable rows from the MemRowset

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Don't output unobservable rows from the MemRowset .. Don't output unobservable rows from the MemRowset In some rare cases we might have a row

[kudu-CR] Add Reinserts to tablet history gc-itest

2016-11-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Add Reinserts to tablet_history_gc-itest .. Add Reinserts to tablet_history_gc-itest This adds Reinserts as a new operation to

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-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/5300 to look at the new patch set (#15). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absense of writes .. Patch Set 21: (29 comments)

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

2016-12-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/5240 to look at the new patch set (#24). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

2016-12-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/5240 to look at the new patch set (#25). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

2016-12-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/5240 to look at the new patch set (#23). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

[kudu-CR] KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 15: (14 comments)

[kudu-CR] WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

2016-12-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/5240 to look at the new patch set (#22). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes

<    2   3   4   5   6   7   8   9   10   11   >