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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin 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: (10 comments)

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

2018-10-23 Thread Hao Hao (Code Review)
Hao Hao 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] [rebalancer] location-aware rebalancer (part 2/n)

2018-10-23 Thread Alexey Serbin (Code Review)
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: Verified+1 Unrelated flake in

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

2018-10-23 Thread Alexey Serbin (Code Review)
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 1/n)

2018-10-23 Thread Alexey Serbin (Code Review)
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 1/n)

2018-10-23 Thread Alexey Serbin (Code Review)
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: Verified+1 Unrelated flake in

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

2018-10-23 Thread Alexey Serbin (Code Review)
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: Verified+1 Unrelated flake in

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

2018-10-23 Thread Alexey Serbin (Code Review)
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: Verified+1 Unrelated flake in

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11746 ) Change subject: [rebalancer] location-aware rebalancer (part 5/n) .. Removed reviewer Kudu Jenkins with the following votes:

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

2018-10-23 Thread Alexey Serbin (Code Review)
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: Verified+1 Unrelated flake in

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11745 ) Change subject: [rebalancer] location-aware rebalancer (part 4/n) .. Removed reviewer Kudu Jenkins with the following votes:

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

2018-10-23 Thread Adar Dembo (Code Review)
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 (#5). Change subject: catalog_manager: adjust maximum tablet count at table creation time

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

2018-10-23 Thread Dan Burkert (Code Review)
Dan Burkert 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 5: Code-Review+1 -- To view, visit

[kudu-CR] WIP: Add mini Sentry to the external mini cluster

2018-10-23 Thread Dan Burkert (Code Review)
Hello Andrew Wong, Hao Hao, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/11756 to review the following change. Change subject: WIP: Add mini Sentry to the external mini cluster .. WIP:

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

2018-10-23 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11748 to look at the new patch set (#2). Change subject: [rebalancer] location-aware rebalancer (part 7/n) ..

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

2018-10-23 Thread Alexey Serbin (Code Review)
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: (5 comments)

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

2018-10-23 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11747 to look at the new patch set (#2). Change subject: [rebalancer] location-aware rebalancer (part 6/n)

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11747 ) Change subject: [rebalancer] location-aware rebalancer (part 6/n) .. Patch Set 1: (5 comments)

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin 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 5: Code-Review+2 -- To view, visit

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

2018-10-23 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11716 ) Change subject: catalog_manager: adjust maximum tablet count at table creation time .. catalog_manager: adjust maximum tablet

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

2018-10-23 Thread Alexey Serbin (Code Review)
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] [rebalancer] location-aware rebalancer (part 7/n)

2018-10-23 Thread Alexey Serbin (Code Review)
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 2: Verified+1 Unrelated flake in

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Patch Set 2: (4 comments)

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11759 to look at the new patch set (#2). Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11759 Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. KUDU-1678: Race during abort of pending

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

2018-10-23 Thread Alexey Serbin (Code Review)
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: (2 comments)

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Patch Set 2: Code-Review+2 -- To view,

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Will Berkeley (Code Review)
Will Berkeley has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Removed reviewer Kudu Jenkins with

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 11: (1 comment)

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 2: (1 comment)

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

2018-10-23 Thread Will Berkeley (Code Review)
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 2: Code-Review+2 -- To view, visit

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

2018-10-23 Thread Will Berkeley (Code Review)
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 2: Code-Review+2 -- To view, visit

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

2018-10-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 6: (1 comment)

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

2018-10-23 Thread Hao Hao (Code Review)
Hao Hao 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: (1 comment)

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

2018-10-23 Thread Hao Hao (Code Review)
Hao Hao 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: (4 comments)

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Patch Set 1: Code-Review+2 (2 comments)

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. Patch Set 2: Verified+1 -- To view,

[kudu-CR] KUDU-1678: Race during abort of pending operations during raft shutdown

2018-10-23 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11759 ) Change subject: KUDU-1678: Race during abort of pending operations during raft shutdown .. KUDU-1678: Race during abort of

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11761 Change subject: [rebalancer] location-aware rebalancer (part 8/n) .. [rebalancer] location-aware rebalancer (part 8/n) Updated

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

2018-10-23 Thread Fengling Wang (Code Review)
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: (7 comments)

[kudu-CR] [master] update placement logic for RF % 2 == 0

2018-10-23 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11763 to look at the new patch set (#2). Change subject: [master] update placement logic for RF % 2 == 0

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

2018-10-23 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, 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 (#4). Change subject: [location_awareness] Add ts location to TSInfoPB

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

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 6: (1 comment)

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

2018-10-23 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, 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 (#5). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [master] update placement logic for RF % 2 == 0

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11763 Change subject: [master] update placement logic for RF % 2 == 0 .. [master] update placement logic for RF % 2 == 0 Updated replica

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

2018-10-23 Thread Fengling Wang (Code Review)
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 (#3). Change subject: [location_awareness] Add ts location to TSInfoPB

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

2018-10-23 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 2: (1 comment)

[kudu-CR] [sentry] SentryAuthzProvider

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11659 ) Change subject: [sentry] SentryAuthzProvider .. Patch Set 3: (4 comments)