[kudu-CR] Test for bug in exactly-once during tablet bootstrap

2016-12-13 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 (#3). Change subject: Test for bug in exactly-once during tablet bootstrap .. Test for

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

2016-12-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: WIP Fix exactly_once bug in tablet bootstrap .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/5432 To unsubscribe, visit http://gerrit.cloudera.org:8

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

2016-12-09 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: I figured out the bug. Will post a fix soon -- To view, visit http://gerrit.cloudera.org

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

2016-12-09 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: After digging around a bit more. I think the problem is a bit more insidious than us just

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

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 [delete_table-test] deleted-while-in-scan test .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5345 To unsubscri

[kudu-CR] KUDU-1753 continue scan if tablet is being deleted

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 continue scan if tablet is being deleted .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5346 To unsubscribe, vi

[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 http://gerrit.cloude

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

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5432 Change subject: WIP Fix exactly_once bug in tablet bootstrap .. WIP Fix exactly_once bug in tablet bootstrap Change-Id: Ia33e6a937

[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] Fix linked list-test flakiness

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5424 Change subject: Fix linked_list-test flakiness .. Fix linked_list-test flakiness A recent patch changed this test to do a snapshot

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

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 [delete_table-test] deleted-while-in-scan test .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5345/7/src/kudu/integration-tests

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

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 [delete_table-test] deleted-while-in-scan test .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5345/7/src/kudu/integration-tests

[kudu-CR] Remove the clock from MvccManager

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/5326 to look at the new patch set (#6). Change subject: Remove the clock from MvccManager .. Remove the clock from MvccM

[kudu-CR] Remove the clock from MvccManager

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/5326 to look at the new patch set (#5). Change subject: Remove the clock from MvccManager .. Remove the clock from MvccM

[kudu-CR] KUDU-1753 continue scan if tablet is being deleted

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 continue scan if tablet is being deleted .. Patch Set 6: (2 comments) lgtm, excepting nits. see my comment on the test patch though. http://ger

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

2016-12-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1753 [delete_table-test] deleted-while-in-scan test .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/5345/7/src/kudu/integration-tests

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

2016-12-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (part 5) Correct safe time advancement This patch fixes safe time advancement in gener

[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 34: Verified+1 overriding jenkins, all tests passed but debug build failed due to: 05:06:31 A

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

2016-12-07 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/5240 to look at the new patch set (#34). Change subject: KUDU-798 (part 5) Correct safe time advancement ..

[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 (#33). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (pa

[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 32: (35 comments) http://gerrit.cloudera.org:8080/#/c/5240/32//COMMIT_MSG Commit Message: PS

[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 30: I won't address any more tidy bot nits that aren't part of the changes in this patch. Its

[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 (#32). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (pa

[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 30: (1 comment) http://gerrit.cloudera.org:8080/#/c/5240/30/src/kudu/tserver/tablet_service.c

[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 (#31). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (pa

[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: PS

[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 (pa

[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 (#29). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (pa

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

2016-12-07 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 5: ha, my bad. Thought this was an old test that was disabled due to flaky

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

2016-12-07 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 5: Did you make sure the test is not flaky anymore? -- To view, visit ht

[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: (3 comments) http://gerrit.cloudera.org:8080/#/c/5240/27/src/kudu/consensus/raft_consensu

[kudu-CR] WIP integration test for snapshot scans

2016-12-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: WIP integration test for snapshot scans .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/5375 To unsubscribe, visit http://gerrit.cloudera.org:8080/s

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

2016-12-07 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 -- To view, visit http://gerrit.cloudera.org:8080/5305 To unsubscribe, visit http://ger

[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 (#28). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (pa

[kudu-CR] Silence gcc warning on TimeManager::GetSafeTimeUnlocked()

2016-12-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Silence gcc warning on TimeManager::GetSafeTimeUnlocked() .. Silence gcc warning on TimeManager::GetSafeTimeUnlocked() gcc is issuing this warning

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

2016-12-06 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 27: (1 comment) http://gerrit.cloudera.org:8080/#/c/5240/27/src/kudu/consensus/raft_consensus

[kudu-CR] Silence gcc warning on TimeManager::GetSafeTimeUnlocked()

2016-12-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5381 Change subject: Silence gcc warning on TimeManager::GetSafeTimeUnlocked() .. Silence gcc warning on TimeManager::GetSafeTimeUnlocke

[kudu-CR] WIP integration test for snapshot scans

2016-12-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/5375 to look at the new patch set (#8). Change subject: WIP integration test for snapshot scans .. WIP integration test

[kudu-CR] WIP integration test for snapshot scans

2016-12-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/5375 to look at the new patch set (#7). Change subject: WIP integration test for snapshot scans .. WIP integration test

[kudu-CR] WIP integration test for snapshot scans

2016-12-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/5375 to look at the new patch set (#6). Change subject: WIP integration test for snapshot scans .. WIP integration test

[kudu-CR] WIP integration test for snapshot scans

2016-12-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/5375 to look at the new patch set (#5). Change subject: WIP integration test for snapshot scans .. WIP integration test

[kudu-CR] WIP integration test for snapshot scans

2016-12-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/5375 to look at the new patch set (#4). Change subject: WIP integration test for snapshot scans .. WIP integration test

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

2016-12-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/5240 to look at the new patch set (#27). Change subject: KUDU-798 (part 5) Correct safe time advancement .. KUDU-798 (pa

[kudu-CR] WIP integration test for snapshot scans

2016-12-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/5375 to look at the new patch set (#3). Change subject: WIP integration test for snapshot scans .. WIP integration test

[kudu-CR] [NOT FOR REVIEW WIP] Design doc for repeatable reads

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/5229 to look at the new patch set (#10). Change subject: [NOT FOR REVIEW WIP] Design doc for repeatable reads .. [NOT FO

[kudu-CR] Leader leases

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: Leader leases .. Abandoned oops pushed some old half-baked stuff -- To view, visit http://gerrit.cloudera.org:8080/5376 To unsubscribe, visit http://gerrit.cloude

[kudu-CR] Leader leases

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5376 Change subject: Leader leases .. Leader leases Change-Id: I7776ad5c65f9c79a7437d8226b59398762dbd6d4 --- M src/kudu/consensus/conse

[kudu-CR] Remove the clock from MvccManager

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/5326 to look at the new patch set (#4). Change subject: Remove the clock from MvccManager .. Remove the clock from MvccM

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

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/5305 to look at the new patch set (#4). Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time ..

[kudu-CR] WIP integration test for snapshot scans

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/5375 to look at the new patch set (#2). Change subject: WIP integration test for snapshot scans .. WIP integration test

[kudu-CR] Output more info when a snapshot scan fails due to history gc

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/5374 to look at the new patch set (#4). Change subject: Output more info when a snapshot scan fails due to history gc ..

[kudu-CR] Output more info when a snapshot scan fails due to history gc

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Output more info when a snapshot scan fails due to history gc .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5374/2/src/kudu/tserver/tab

[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 submitted this change and it was merged. Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. KUDU-798 (part 4) Add a TimeManager to manage safe time advancement This p

[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 18: Code-Review+2 keeping todd's previous +2 since I fixed the nit -- T

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

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

[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 16: (2 comments) http://gerrit.cloudera.org:8080/#/c/5300/16/src/kudu/co

[kudu-CR] Output more info when a snapshot scan fails due to history gc

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5374 Change subject: Output more info when a snapshot scan fails due to history gc .. Output more info when a snapshot scan fails due to

[kudu-CR] WIP integration test for snapshot scans

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5375 Change subject: WIP integration test for snapshot scans .. WIP integration test for snapshot scans This makes executions of TestWo

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/con

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/con

[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 (#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 (#23). Change subject: WIP: KUDU-798 (part 5) Safe time advancement in the absence of writes .

[kudu-CR] KUDU-1622. result tracker: respond to RPCs outside of the lock

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1622. result_tracker: respond to RPCs outside of the lock .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5359 To uns

[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 .

[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 (#16). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[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) http://gerrit.cloudera.org:8080/#/c/5300/15/src/kudu/c

[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) http://gerrit.cloudera.org:8080/#/c/5240/21//COMMIT_

[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 14: (12 comments) http://gerrit.cloudera.org:8080/#/c/5300/14/src/kudu/c

[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] 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 (#14). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[kudu-CR] exactly once rpc-test: fix gc stress test flakiness

2016-12-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: exactly_once_rpc-test: fix gc stress test flakiness .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5358 To unsubscribe, v

[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 (#13). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[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 (#12). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[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 (#11). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

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

2016-12-04 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 (#10). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

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

2016-12-04 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 (#9). Change subject: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement

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

2016-12-04 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 7: (3 comments) http://gerrit.cloudera.org:8080/#/c/5300/7/src/kudu

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

2016-12-04 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement .. Patch Set 8: (19 comments) http://gerrit.cloudera.org:8080/#/c/5300/8/src/kud

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

2016-12-04 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 (#8). Change subject: WIP: KUDU-798 (part 4) Add a TimeManager to manage safe time advancement ...

[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 of

[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 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) http://gerrit.cloudera.org:8080/#/c/5349/8/src/kudu/integration-tests/raft_consensus-

[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 test

[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) http://gerrit.cloudera.org:8080/#/c/5349/8/src/kudu/integration-tests/raft_consensus-

[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 (https://gerrit.cloudera.org

[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 test

[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 7: (10 comments) http://gerrit.cloudera.org:8080/#/c/5349/7/src/kudu/integration-tests/cluster_verif

[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 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 (#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 test

[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 test

[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 test

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

<    6   7   8   9   10   11   12   13   14   15   >