[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11748 ) Change subject: [rebalancer] location-aware rebalancer (part 7/n) .. Patch Set 1: Verified+1 Unrelated flake in

[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11748 ) Change subject: [rebalancer] location-aware rebalancer (part 7/n) .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] catalog manager: adjust maximum tablet count at table creation time

Hello Will Berkeley, Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11716 to look at the new patch set (#4). Change subject: catalog_manager: adjust maximum tablet count at table creation time

[kudu-CR] catalog manager: adjust maximum tablet count at table creation time

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11716 ) Change subject: catalog_manager: adjust maximum tablet count at table creation time .. Patch Set 3: (4 comments)

[kudu-CR] WIP KUDU-2542: add initial authorization token impl

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: WIP KUDU-2542: add initial authorization token impl .. Patch Set 1: (15 comments)

[kudu-CR] WIP authz: verify tokens on scans

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11753 ) Change subject: WIP authz: verify tokens on scans .. Patch Set 1: (3 comments) Just responding

[kudu-CR] [rebalancer] location-aware rebalancer (part 1/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11549 ) Change subject: [rebalancer] location-aware rebalancer (part 1/n) .. Patch Set 7: (12 comments)

[kudu-CR] [rebalancer] location-aware rebalancer (part 2/n)

Hello Will Berkeley, Tidy Bot, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11743 to look at the new patch set (#2). Change subject: [rebalancer] location-aware rebalancer (part 2/n) ..

[kudu-CR] [rebalancer] location-aware rebalancer (part 1/n)

Hello Will Berkeley, Tidy Bot, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11549 to look at the new patch set (#7). Change subject: [rebalancer] location-aware rebalancer (part 1/n) ..

[kudu-CR] [rebalancer] location-aware rebalancer (part 4/n)

Hello Will Berkeley, Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11745 to look at the new patch set (#2). Change subject: [rebalancer] location-aware rebalancer (part 4/n)

[kudu-CR] [rebalancer] location-aware rebalancer (part 3/n)

Hello Will Berkeley, Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11744 to look at the new patch set (#2). Change subject: [rebalancer] location-aware rebalancer (part 3/n)

[kudu-CR] [rebalancer] location-aware rebalancer (part 4/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11745 ) Change subject: [rebalancer] location-aware rebalancer (part 4/n) .. Patch Set 2: (3 comments)

[kudu-CR] [rebalancer] location-aware rebalancer (part 3/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11744 ) Change subject: [rebalancer] location-aware rebalancer (part 3/n) .. Patch Set 2: (3 comments)

[kudu-CR] [rebalancer] location-aware rebalancer (part 5/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11746 ) Change subject: [rebalancer] location-aware rebalancer (part 5/n) .. Patch Set 2: (1 comment)

[kudu-CR] [rebalancer] location-aware rebalancer (part 2/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11743 ) Change subject: [rebalancer] location-aware rebalancer (part 2/n) .. Patch Set 2: (1 comment)

[kudu-CR] [rebalancer] location-aware rebalancer (part 5/n)

Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11746 to look at the new patch set (#2). Change subject: [rebalancer] location-aware rebalancer (part 5/n)

[kudu-CR] [rebalancer] location-aware rebalancer (part 3/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11744 ) Change subject: [rebalancer] location-aware rebalancer (part 3/n) .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [sentry] improve SentryAction

Hao Hao has removed a vote on this change. Change subject: [sentry] improve SentryAction .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11720 To unsubscribe, visit

[kudu-CR] [sentry] improve SentryAction

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11720 ) Change subject: [sentry] improve SentryAction .. Patch Set 5: Verified+1 Flaky unrelated

[kudu-CR] [sentry] SentryAuthzProvider

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] SentryAuthzProvider .. Patch Set 3: Verified+1 Unrelated flaky

[kudu-CR] [sentry] SentryAuthzProvider

Hao Hao has removed a vote on this change. Change subject: [sentry] SentryAuthzProvider .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/11659 To unsubscribe, visit

[kudu-CR] [location awareness] Add ts location to both client and internal client

Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#5). Change subject: [location_awareness] Add ts location to both client and internal client

[kudu-CR] benchmarks: work around missing node density-itest timing measurements

Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11710 ) Change subject: benchmarks: work around missing node_density-itest timing measurements .. benchmarks: work around missing

[kudu-CR] [sentry] SentryAuthzProvider

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] SentryAuthzProvider .. Patch Set 3: (3 comments)

[kudu-CR] [rebalancer] location-aware rebalancer (part 2/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11743 ) Change subject: [rebalancer] location-aware rebalancer (part 2/n) .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [rebalancer] location-aware rebalancer (part 1/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11549 ) Change subject: [rebalancer] location-aware rebalancer (part 1/n) .. Patch Set 6: (12 comments)

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Patch Set 1: (1 comment)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: Added a test to verify that the TS location is

[kudu-CR] [sentry] improve SentryAction

Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11720 ) Change subject: [sentry] improve SentryAction .. [sentry] improve SentryAction A follow-up to Adar's suggestions in

[kudu-CR] [sentry] Fill out more sentry client API

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11657 ) Change subject: [sentry] Fill out more sentry client API .. Patch Set 5: (1 comment)

[kudu-CR] [sentry] SentryAuthzProvider

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] SentryAuthzProvider .. Patch Set 3: (22 comments) http://gerrit.cloudera.org:8080/#/c/11659/3//COMMIT_MSG

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. De-flake ASAN

[kudu-CR] [rebalancer] location-aware rebalancer (part 7/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11748 ) Change subject: [rebalancer] location-aware rebalancer (part 7/n) .. Patch Set 1: (5 comments)

[kudu-CR] [sentry] improve SentryAction

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11720 ) Change subject: [sentry] improve SentryAction .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/11720 To

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Patch Set 1: (1 comment)

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11737 to look at the new patch set (#2). Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (1 comment)

[kudu-CR] [rebalancer] location-aware rebalancer (part 4/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11745 ) Change subject: [rebalancer] location-aware rebalancer (part 4/n) .. Patch Set 1: (3 comments) Thanks for splitting the patches up.

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Patch Set 2: Verified+1 Unrelated failure of a few

[kudu-CR] [location awareness] Add ts location to both client and internal client

Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#6). Change subject: [location_awareness] Add ts location to both client and internal client

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11727 to look at the new patch set (#2). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (2 comments)

[kudu-CR] [rebalancer] location-aware rebalancer (part 6/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11747 ) Change subject: [rebalancer] location-aware rebalancer (part 6/n) .. Patch Set 1: (4 comments)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (1 comment) I'm thinking maybe the test is better

[kudu-CR] [sentry] Fill out more sentry client API

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11657 ) Change subject: [sentry] Fill out more sentry client API .. Patch Set 5: (1 comment)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (6 comments)

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Patch Set 1: (2 comments)

[kudu-CR] [rebalancer] location-aware rebalancer (part 5/n)

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11746 ) Change subject: [rebalancer] location-aware rebalancer (part 5/n) .. Patch Set 1: (1 comment) LGTM. I think I also reviewed this

[kudu-CR] WIP authz: verify tokens on writes

Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11754 Change subject: WIP authz: verify tokens on writes .. WIP authz: verify tokens on writes WIP: need to decode the op types and check the

[kudu-CR] WIP authz: verify tokens on scans

Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11753 Change subject: WIP authz: verify tokens on scans .. WIP authz: verify tokens on scans WIP: add tests Scans require privileges on the

[kudu-CR] WIP KUDU-2542: add initial authorization token impl

Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11750 Change subject: WIP KUDU-2542: add initial authorization token impl .. WIP KUDU-2542: add initial authorization token impl WIP: add

[kudu-CR] WIP KUDU-2543: pass around default authz tokens

Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11751 Change subject: WIP KUDU-2543: pass around default authz tokens .. WIP KUDU-2543: pass around default authz tokens WIP: add tests WIP:

[kudu-CR] WIP authz: authorize ListTablets

Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11752 Change subject: WIP authz: authorize ListTablets .. WIP authz: authorize ListTablets WIP: add tests See the comment in tablet_service.h

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Will Berkeley has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Removed reviewer Kudu Jenkins with the

[kudu-CR] [rebalancer] location-aware rebalancer (part 1/n)

Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11549 ) Change subject: [rebalancer] location-aware rebalancer (part 1/n) .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] [rebalancer] location-aware rebalancer (part 2/n)

Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11743 ) Change subject: [rebalancer] location-aware rebalancer (part 2/n) .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] [rebalancer] location-aware rebalancer (part 2/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11743 ) Change subject: [rebalancer] location-aware rebalancer (part 2/n) .. Patch Set 1: Verified+1 Unrelated flake in

[kudu-CR] [sentry] Fill out more sentry client API

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11657 ) Change subject: [sentry] Fill out more sentry client API .. Patch Set 5: (7 comments)

[kudu-CR] [sentry] improve SentryAction

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11720 ) Change subject: [sentry] improve SentryAction .. Patch Set 5: (1 comment)

[kudu-CR] [sentry] improve SentryAction

Hello Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11720 to look at the new patch set (#5). Change subject: [sentry] improve SentryAction ..

[kudu-CR] [sentry] SentryAuthzProvider

Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11659 to look at the new patch set (#3). Change subject: [sentry] SentryAuthzProvider .. [sentry]

[kudu-CR] [rebalancer] location-aware rebalancer (part 1/n)

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11549 ) Change subject: [rebalancer] location-aware rebalancer (part 1/n) .. Patch Set 6: Verified+1 Unrelated flake in

[kudu-CR] De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11737 ) Change subject: De-flake ASAN TestSimultaneousLeaderTransferAndAbruptStepdown .. Patch Set 1: (1 comment)

[kudu-CR] WIP KUDU-2543: pass around default authz tokens

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11751 ) Change subject: WIP KUDU-2543: pass around default authz tokens .. Patch Set 1: (1 comment)

[kudu-CR] WIP KUDU-2542: add initial authorization token impl

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: WIP KUDU-2542: add initial authorization token impl .. Patch Set 1: (2 comments)

[kudu-CR] WIP authz: verify tokens on scans

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11753 ) Change subject: WIP authz: verify tokens on scans .. Patch Set 1: (8 comments)

[kudu-CR] WIP authz: verify tokens on writes

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11754 ) Change subject: WIP authz: verify tokens on writes .. Patch Set 1: (1 comment)

[kudu-CR] WIP KUDU-2542: add initial authorization token impl

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11750 ) Change subject: WIP KUDU-2542: add initial authorization token impl .. Patch Set 1: (4 comments)

[kudu-CR] WIP KUDU-2543: pass around default authz tokens

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11751 ) Change subject: WIP KUDU-2543: pass around default authz tokens .. Patch Set 1: (17 comments)

[kudu-CR] WIP authz: authorize ListTablets

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11752 ) Change subject: WIP authz: authorize ListTablets .. Patch Set 1: LGTM -- To view, visit http://gerrit.cloudera.org:8080/11752 To

[kudu-CR] [location awareness] Add ts location to TSInfoPB

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (4 comments)