[kudu-CR] KUDU-2584: Prevent flaky off-by-one errors in backup tests

2018-11-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11815 ) Change subject: KUDU-2584: Prevent flaky off-by-one errors in backup tests .. Patch Set 2: (2 comments)

[kudu-CR] Limit number of rowsets in compaction selection to 32 in TSAN mode

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11885 ) Change subject: Limit number of rowsets in compaction selection to 32 in TSAN mode .. Patch Set 1: (2 comments)

[kudu-CR] Limit number of rowsets in compaction selection to 32 in TSAN mode

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11885 ) Change subject: Limit number of rowsets in compaction selection to 32 in TSAN mode .. Patch Set 1: (1 comment)

[kudu-CR] Limit number of rowsets in compaction selection to 32 in TSAN mode

2018-11-05 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11885 ) Change subject: Limit number of rowsets in compaction selection to 32 in TSAN mode .. Patch Set 1: (2 comments)

[kudu-CR] Limit number of rowsets in compaction selection to 32 in TSAN mode

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11885 ) Change subject: Limit number of rowsets in compaction selection to 32 in TSAN mode .. Patch Set 1: (3 comments)

[kudu-CR] Disable rowset compaction in TestUndoDeltaBlockGc

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11884 ) Change subject: Disable rowset compaction in TestUndoDeltaBlockGc .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-05 Thread Michael Ho (Code Review)
Hello Kudu Jenkins, Sailesh Mukil, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11681 to look at the new patch set (#4). Change subject: Add "network_plane" as part of ConnectionId

[kudu-CR] KUDU-1918 Prevent hijacking of scanner IDs

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/6348 ) Change subject: KUDU-1918 Prevent hijacking of scanner IDs .. Patch Set 7: Code-Review+2 (5 comments) LGTM, just a few nits.

[kudu-CR] Limit number of rowsets in compaction selection to 32 in TSAN mode

2018-11-05 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11885 Change subject: Limit number of rowsets in compaction selection to 32 in TSAN mode .. Limit number of rowsets in compaction selection

[kudu-CR] Disable rowset compaction in TestUndoDeltaBlockGc

2018-11-05 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11884 Change subject: Disable rowset compaction in TestUndoDeltaBlockGc .. Disable rowset compaction in TestUndoDeltaBlockGc In a follow-up

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-05 Thread Michael Ho (Code Review)
Hello Kudu Jenkins, Sailesh Mukil, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11681 to look at the new patch set (#3). Change subject: Add "network_plane" as part of ConnectionId

[kudu-CR] Add "network plane" as part of ConnectionId

2018-11-05 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11681 ) Change subject: Add "network_plane" as part of ConnectionId .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2584: Prevent flaky off-by-one errors in backup tests

2018-11-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11815 ) Change subject: KUDU-2584: Prevent flaky off-by-one errors in backup tests .. Patch Set 2: (4 comments)

[kudu-CR] [tools] reference comparison mode for rebalancing algo tests

2018-11-05 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11870 to look at the new patch set (#3). Change subject: [tools] reference comparison mode for rebalancing algo tests

[kudu-CR] [compaction] KUDU-1400: Improve rowset compaction policy to consider merging small DRSs

2018-11-05 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11869 ) Change subject: [compaction] KUDU-1400: Improve rowset compaction policy to consider merging small DRSs .. Patch Set 3: > I'm not

[kudu-CR] [tools] reference comparison mode for rebalancing algo tests

2018-11-05 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11870 to look at the new patch set (#2). Change subject: [tools] reference comparison mode for rebalancing algo tests

[kudu-CR] Add TS UUID to alter table-randomized-test "restart TS" step

2018-11-05 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11877 ) Change subject: Add TS UUID to alter_table-randomized-test "restart TS" step .. Add TS UUID to alter_table-randomized-test

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. [sentry] add AuthzProvider This commit adds a high-level abstraction which handles

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 10: (1 comment)

[kudu-CR] Add TS UUID to alter table-randomized-test "restart TS" step

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11877 ) Change subject: Add TS UUID to alter_table-randomized-test "restart TS" step .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 14: Code-Review+2 (1 comment)

[kudu-CR] Add TS UUID to alter table-randomized-test "restart TS" step

2018-11-05 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11877 ) Change subject: Add TS UUID to alter_table-randomized-test "restart TS" step .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [examples] Add basic Spark example (scala)

2018-11-05 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example (scala) .. [examples] Add basic Spark example (scala) This patch adds a

[kudu-CR] [examples] Add basic Spark example (scala)

2018-11-05 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11788 ) Change subject: [examples] Add basic Spark example (scala) .. Patch Set 12: Code-Review+2 -- To view, visit

[kudu-CR] Add TS UUID to alter table-randomized-test "restart TS" step

2018-11-05 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11877 Change subject: Add TS UUID to alter_table-randomized-test "restart TS" step .. Add TS UUID to alter_table-randomized-test "restart TS"

[kudu-CR] KUDU-1918 Prevent hijacking of scanner IDs

2018-11-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/6348 ) Change subject: KUDU-1918 Prevent hijacking of scanner IDs .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 14: (1 comment)

[kudu-CR] KUDU-2584: Prevent flaky off-by-one errors in backup tests

2018-11-05 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11815 ) Change subject: KUDU-2584: Prevent flaky off-by-one errors in backup tests .. Patch Set 2: Verified+1 Failure was a know

[kudu-CR] KUDU-2584: Prevent flaky off-by-one errors in backup tests

2018-11-05 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: KUDU-2584: Prevent flaky off-by-one errors in backup tests .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11815 To

[kudu-CR] [sentry] add AuthzProvider

2018-11-05 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] add AuthzProvider .. Patch Set 14: Code-Review+1 (1 comment)

[kudu-CR] KUDU-2584: Prevent flaky off-by-one errors in backup tests

2018-11-05 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/11815 ) Change subject: KUDU-2584: Prevent flaky off-by-one errors in backup tests .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-2584: Prevent flaky off-by-one errors in backup tests

2018-11-05 Thread Grant Henke (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11815 to look at the new patch set (#2). Change subject: KUDU-2584: Prevent flaky off-by-one errors in backup tests