[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-10-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 9: Code-Review+2 -- To

[kudu-CR] KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator

2018-10-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11395 ) Change subject: KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator .. Patch Set 10: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator

2018-10-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11395 ) Change subject: KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator .. Patch Set 9: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator

2018-10-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11395 ) Change subject: KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator .. Patch Set 9: +2ing but might make sense to add

[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-10-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 8: Code-Review+2 -- To

[kudu-CR] KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator

2018-10-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11395 ) Change subject: KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator .. Patch Set 8: (13 comments)

[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-10-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 7: Code-Review+2 -- To

[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-09-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 6: Code-Review+2 -- To

[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-09-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 5: Code-Review+2 -- To

[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-09-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 4: (1 comment)

[kudu-CR] hybrid clock: restore SleepFor in WaitUntilAfterLocally

2018-09-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11495 ) Change subject: hybrid_clock: restore SleepFor in WaitUntilAfterLocally .. Patch Set 2: I guess I should have +2 before the

[kudu-CR] hybrid clock: restore SleepFor in WaitUntilAfterLocally

2018-09-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11495 ) Change subject: hybrid_clock: restore SleepFor in WaitUntilAfterLocally .. hybrid_clock: restore SleepFor in

[kudu-CR] hybrid clock: restore SleepFor in WaitUntilAfterLocally

2018-09-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11495 ) Change subject: hybrid_clock: restore SleepFor in WaitUntilAfterLocally .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator

2018-09-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11395 ) Change subject: KUDU-686 (part 2/2): use DeltaPreparer in DeltaFileIterator .. Patch Set 4: (5 comments) I think we should

[kudu-CR] KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer

2018-09-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/11394 ) Change subject: KUDU-686 (part 1/2): decompose guts of DMSIterator into DeltaPreparer .. Patch Set 4: (6 comments)

[kudu-CR] WIP: Atomicize OpId and Timestamp assignment

2018-07-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/11070 to look at the new patch set (#6). Change subject: WIP: Atomicize OpId and Timestamp assignment .. WIP: Atomicize

[kudu-CR] WIP: Atomicize OpId and Timestamp assignment

2018-07-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/11070 to look at the new patch set (#5). Change subject: WIP: Atomicize OpId and Timestamp assignment .. WIP: Atomicize

[kudu-CR] WIP: Atomicize OpId and Timestamp assignment

2018-07-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/11070 to look at the new patch set (#4). Change subject: WIP: Atomicize OpId and Timestamp assignment .. WIP: Atomicize

[kudu-CR] [tablet] Avoid compactions in the past

2018-07-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8965 ) Change subject: [tablet] Avoid compactions in the past .. Patch Set 11: (1 comment)

[kudu-CR] WIP: Atomicize OpId and Timestamp assignment

2018-07-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/11070 to look at the new patch set (#3). Change subject: WIP: Atomicize OpId and Timestamp assignment .. WIP: Atomicize

[kudu-CR] WIP: Atomicize OpId and Timestamp assignment

2018-07-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/11070 to look at the new patch set (#2). Change subject: WIP: Atomicize OpId and Timestamp assignment .. WIP: Atomicize

[kudu-CR] WIP: Atomicize OpId and Timestamp assignment

2018-07-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11070 Change subject: WIP: Atomicize OpId and Timestamp assignment .. WIP: Atomicize OpId and Timestamp assignment This is a simpler

[kudu-CR] WIP: Simplify the previous simplification of opid/timestamp assignment

2018-07-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11069 Change subject: WIP: Simplify the previous simplification of opid/timestamp assignment .. WIP: Simplify the previous

[kudu-CR] WIP: Checking if missing this check is somehow the reason for the timeouts

2018-07-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11065 Change subject: WIP: Checking if missing this check is somehow the reason for the timeouts .. WIP: Checking if missing this

[kudu-CR] Add a way to pin clean time advancement

2018-07-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has removed a vote on this change. Change subject: Add a way to pin clean time advancement .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/9641 To unsubscribe,

[kudu-CR] Add a way to pin clean time advancement

2018-07-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9641 ) Change subject: Add a way to pin clean time advancement .. Patch Set 7: Verified+1 (24 comments) Oops had some unposted old

[kudu-CR] [rpc tests] Reduce flakyness by setting a longer keepalive in TSAN

2018-07-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. ( http://gerrit.cloudera.org:8080/6289 ) Change subject: [rpc tests] Reduce flakyness by setting a longer keepalive in TSAN .. Abandoned I no longer have a patch around I can

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

2018-07-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. ( http://gerrit.cloudera.org:8080/5229 ) Change subject: [NOT FOR REVIEW WIP] Design doc for repeatable reads .. Abandoned no longer relevant since was largely fixed in other

[kudu-CR] [flaky tests] Increase the result ttl for ExactlyOnceRpcTest.TestExactlyOnceSemanticsGarbageCollectionStressTest

2018-07-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. ( http://gerrit.cloudera.org:8080/4644 ) Change subject: [flaky tests] Increase the result ttl for ExactlyOnceRpcTest.TestExactlyOnceSemanticsGarbageCollectionStressTest ..

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-07-24 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#28). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] Add a way to pin clean time advancement

2018-07-24 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Hao Hao, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9641 to look at the new patch set (#7). Change subject: Add a way to pin clean time advancement

[kudu-CR] exactly once writes-itest for faulty disks

2018-07-24 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8388 ) Change subject: exactly_once_writes-itest for faulty disks .. Patch Set 8: (7 comments)

[kudu-CR] exactly once writes-itest for faulty disks

2018-07-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8388 ) Change subject: exactly_once_writes-itest for faulty disks .. Patch Set 7: this still worth merging? -- To view, visit

[kudu-CR] [python] : Add Pandas Support to Scanner

2018-06-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10809 ) Change subject: [python] : Add Pandas Support to Scanner .. Patch Set 4: (1 comment)

[kudu-CR] parse metrics log: update to the new format

2018-06-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10693 ) Change subject: parse_metrics_log: update to the new format .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: consensus: separate storage from ConsensusMetadata .. Patch Set 4: (5 comments) I guess andrew's point is that

[kudu-CR] KUDU-2441 - [python] Enable configuration of mutation buffer

2018-06-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10674 ) Change subject: KUDU-2441 - [python] Enable configuration of mutation buffer .. Patch Set 4: (3 comments)

[kudu-CR] [examples] KUDU-2382: Add Kudu Basic Python Example

2018-06-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10794 ) Change subject: [examples] KUDU-2382: Add Kudu Basic Python Example .. Patch Set 4: (3 comments)

[kudu-CR] [python] : Add Pandas Support to Scanner

2018-06-25 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10809 ) Change subject: [python] : Add Pandas Support to Scanner .. Patch Set 4: (2 comments)

[kudu-CR] parse metrics log: minor refactoring

2018-06-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10694 ) Change subject: parse_metrics_log: minor refactoring .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] scripts: tool to plot metrics

2018-06-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10695 ) Change subject: scripts: tool to plot metrics .. Patch Set 1: (3 comments)

[kudu-CR] parse metrics log: update to the new format

2018-06-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10693 ) Change subject: parse_metrics_log: update to the new format .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] WIP consensus: remove env usage from ConsensusMetadata

2018-06-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: WIP consensus: remove env usage from ConsensusMetadata .. Patch Set 1: (3 comments)

[kudu-CR] wip tablet: add tmeta manager

2018-06-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10650 ) Change subject: wip tablet: add tmeta manager .. Patch Set 1: (5 comments)

[kudu-CR] WIP: Possible fix for the timeouts

2018-05-25 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10468 to look at the new patch set (#3). Change subject: WIP: Possible fix for the timeouts .. WIP: Possible fix for the

[kudu-CR] WIP: Possible fix for the timeouts

2018-05-24 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10468 to look at the new patch set (#2). Change subject: WIP: Possible fix for the timeouts .. WIP: Possible fix for the

[kudu-CR] WIP: Possible fix for the timeouts

2018-05-21 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10468 Change subject: WIP: Possible fix for the timeouts .. WIP: Possible fix for the timeouts Change-Id:

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 14: Code-Review+2 -- To view, visit

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 9: (1 comment)

[kudu-CR] Fix int overflow GetClockTimeMicros() on macOS

2018-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10371 ) Change subject: Fix int overflow GetClockTimeMicros() on macOS .. Fix int overflow GetClockTimeMicros() on macOS On

[kudu-CR] Fix int overflow GetClockTimeMicros() on macOS

2018-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10371 ) Change subject: Fix int overflow GetClockTimeMicros() on macOS .. Patch Set 1: (2 comments)

[kudu-CR] Fix int overflow GetClockTimeMicros() on macOS

2018-05-10 Thread David Ribeiro Alves (Code Review)
Hello Dan Burkert, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10371 to look at the new patch set (#2). Change subject: Fix int overflow GetClockTimeMicros() on macOS

[kudu-CR] Fix int overflow GetClockTimeMicros() on macOS

2018-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10371 ) Change subject: Fix int overflow GetClockTimeMicros() on macOS .. Patch Set 1: (1 comment)

[kudu-CR] Fix int overflow GetClockTimeMicros() on macOS

2018-05-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10371 Change subject: Fix int overflow GetClockTimeMicros() on macOS .. Fix int overflow GetClockTimeMicros() on macOS On macOS

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-05-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 9: (17 comments)

[kudu-CR] python-client: fix test scan rows string predicate and projection

2018-05-01 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10255 ) Change subject: python-client: fix test_scan_rows_string_predicate_and_projection .. Patch Set 5: Code-Review+2 -- To view,

[kudu-CR] python-client: fix test scan rows string predicate and projection

2018-04-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10255 ) Change subject: python-client: fix test_scan_rows_string_predicate_and_projection .. Patch Set 2: (1 comment)

[kudu-CR] java: prohibit use of a KuduTable from an unassociated KuduClient

2018-04-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7362 ) Change subject: java: prohibit use of a KuduTable from an unassociated KuduClient .. Patch Set 6: Code-Review+2 thanks -- To

[kudu-CR] KUDU-2399: Support IS NULL / IS NOT NULL predicates in Python

2018-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10072 ) Change subject: KUDU-2399: Support IS NULL / IS NOT NULL predicates in Python .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2399: Support IS NULL / IS NOT NULL predicates in Python

2018-04-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10072 ) Change subject: KUDU-2399: Support IS NULL / IS NOT NULL predicates in Python .. Patch Set 4: (1 comment) thanks for the

[kudu-CR] java: prohibit use of a KuduTable from an unassociated KuduClient

2018-04-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7362 ) Change subject: java: prohibit use of a KuduTable from an unassociated KuduClient .. Patch Set 5: (2 comments)

[kudu-CR] cache: reduce contention on MemTracker::Release and Consume

2018-04-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/6597 ) Change subject: cache: reduce contention on MemTracker::Release and Consume .. Patch Set 4: (2 comments)

[kudu-CR] java: prohibit use of a KuduTable from an unassociated KuduClient

2018-04-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7362 ) Change subject: java: prohibit use of a KuduTable from an unassociated KuduClient .. Patch Set 5: (2 comments)

[kudu-CR] WIP Workaround KUDU-2233 when the history was already deleted

2018-04-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10163 ) Change subject: WIP Workaround KUDU-2233 when the history was already deleted .. Patch Set 1: (1 comment)

[kudu-CR] WIP Workaround KUDU-2233 when the history was already deleted

2018-04-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10163 Change subject: WIP Workaround KUDU-2233 when the history was already deleted .. WIP Workaround KUDU-2233 when the history was

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-04-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7221 ) Change subject: Simplify OpId/Timestamp assignment and make it atomic .. Patch Set 24: (7 comments)

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-04-23 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#25). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] Add a way to pin clean time advancement

2018-04-23 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Hao Hao, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9641 to look at the new patch set (#4). Change subject: Add a way to pin clean time advancement

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-04-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 5: (7 comments)

[kudu-CR] KUDU-2399: Support IS NULL / IS NOT NULL predicates in Python

2018-04-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/10072 ) Change subject: KUDU-2399: Support IS NULL / IS NOT NULL predicates in Python .. Patch Set 4: (4 comments)

[kudu-CR] A new Jepsen checker for READ YOUR WRITES scan mode

2018-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9526 ) Change subject: A new Jepsen checker for READ_YOUR_WRITES scan mode .. Patch Set 4: (2 comments) Thanks for this valiant

[kudu-CR] KUDU-2232. java: Remove deprecated call from KuduTable

2018-03-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8749 ) Change subject: KUDU-2232. java: Remove deprecated call from KuduTable .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-16 pt 2: add client-side limits on scanners

2018-03-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9790 ) Change subject: KUDU-16 pt 2: add client-side limits on scanners .. Patch Set 1: Haven't fully reviewed this patch, just looked

[kudu-CR] KUDU-16 pt 1: add server-side limits on scanners

2018-03-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9783 ) Change subject: KUDU-16 pt 1: add server-side limits on scanners .. Patch Set 2: (6 comments)

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-22 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#24). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-22 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#23). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-22 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#22). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-15 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#21). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR](branch-1.7.x) docs: update release notes and scaling guide with new thread information

2018-03-15 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9682 ) Change subject: docs: update release notes and scaling guide with new thread information .. Patch Set 1: Code-Review+2 Not sure

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-15 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7221 ) Change subject: Simplify OpId/Timestamp assignment and make it atomic .. Patch Set 19: > Patch Set 19: Verified-1 > > Build

[kudu-CR] docs: update release notes and scaling guide with new thread information

2018-03-15 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9552 ) Change subject: docs: update release notes and scaling guide with new thread information .. Patch Set 5: Code-Review+2 -- To

[kudu-CR] docs: update release notes and scaling guide with new thread information

2018-03-15 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9552 ) Change subject: docs: update release notes and scaling guide with new thread information .. Patch Set 3: (4 comments)

[kudu-CR] Add a way to pin clean time advancement

2018-03-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/9641 to look at the new patch set (#2). Change subject: Add a way to pin clean time advancement .. Add a way to pin

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-14 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#19). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7221 ) Change subject: Simplify OpId/Timestamp assignment and make it atomic .. Patch Set 17: (1 comment)

[kudu-CR] [release notes]: update notes for RYW read mode

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9634 ) Change subject: [release notes]: update notes for RYW read mode .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-14 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Mike Percy, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#18). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] WIP: Add a way to pin clean time advancement

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9641 Change subject: WIP: Add a way to pin clean time advancement .. WIP: Add a way to pin clean time advancement Change-Id:

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/7221 ) Change subject: Simplify OpId/Timestamp assignment and make it atomic .. Patch Set 8: (36 comments)

[kudu-CR] [release notes]: update notes for RYW read mode

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9634 ) Change subject: [release notes]: update notes for RYW read mode .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1704: add python client support for READ YOUR WRITES mode

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9617 ) Change subject: KUDU-1704: add python client support for READ_YOUR_WRITES mode .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1704: add python client support for READ YOUR WRITES mode

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9617 ) Change subject: KUDU-1704: add python client support for READ_YOUR_WRITES mode .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1704: add python client support for READ YOUR WRITES mode

2018-03-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9617 ) Change subject: KUDU-1704: add python client support for READ_YOUR_WRITES mode .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1704: add python client support for READ YOUR WRITES mode

2018-03-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9617 ) Change subject: KUDU-1704: add python client support for READ_YOUR_WRITES mode .. Patch Set 1: (6 comments)

[kudu-CR] [release notes]: RYW read mode

2018-03-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9594 ) Change subject: [release notes]: RYW read mode .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [release notes]: RYW read mode

2018-03-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9594 ) Change subject: [release notes]: RYW read mode .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1704: add java client support for READ YOUR WRITES mode

2018-03-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8847 ) Change subject: KUDU-1704: add java client support for READ_YOUR_WRITES mode .. Patch Set 10: as discussed offline, ok with

[kudu-CR] KUDU-1704: add java client support for READ YOUR WRITES mode

2018-03-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8847 ) Change subject: KUDU-1704: add java client support for READ_YOUR_WRITES mode .. Patch Set 10: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2093: Document AbstractKuduScannerBuilder limit has no effect

2018-03-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9559 ) Change subject: KUDU-2093: Document AbstractKuduScannerBuilder limit has no effect .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] KUDU-1704: add java client support for READ YOUR WRITES mode

2018-03-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8847 ) Change subject: KUDU-1704: add java client support for READ_YOUR_WRITES mode .. Patch Set 10: lgtm, thanks waiting to get some

  1   2   3   4   5   6   7   8   9   10   >