[kudu-CR] Rename DeadlineTracker to TimeoutTracker

2019-02-07 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12373 to look at the new patch set (#2). Change subject: Rename DeadlineTracker to TimeoutTracker

[kudu-CR] KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

2019-02-07 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12363 to look at the new patch set (#3). Change subject: KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

[kudu-CR] KUDU-1868: Part 1: Add timer-based RPC timeouts

2019-02-07 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12338 to look at the new patch set (#6). Change subject: KUDU-1868: Part 1: Add timer-based RPC timeouts

[kudu-CR] KUDU-2674: [java] Add a Java KuduPartitioner API

2019-02-07 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12275 to look at the new patch set (#5). Change subject: KUDU-2674: [java] Add a Java KuduPartitioner API

[kudu-CR] KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

2019-02-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12363 ) Change subject: KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation .. Patch Set 2: (8 comments)

[kudu-CR] KUDU-2674: [java] Add a Java KuduPartitioner API

2019-02-07 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12275 to look at the new patch set (#6). Change subject: KUDU-2674: [java] Add a Java KuduPartitioner API

[kudu-CR] KUDU-1868: Part 1: Add timer-based RPC timeouts

2019-02-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12338 ) Change subject: KUDU-1868: Part 1: Add timer-based RPC timeouts .. Patch Set 5: (5 comments)

[kudu-CR] KUDU-2676 Support restoring tables over the maximum allowed replicas

2019-02-07 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12372 ) Change subject: KUDU-2676 Support restoring tables over the maximum allowed replicas .. KUDU-2676 Support restoring tables

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

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

[kudu-CR](branch-1.9.x) [rebalancer] location-aware rebalancer (part 10/n)

2019-02-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12395 ) Change subject: [rebalancer] location-aware rebalancer (part 10/n) .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.9.x) [tools] --load imbalance threshold flag for LA rebalancer

2019-02-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12394 ) Change subject: [tools] --load_imbalance_threshold flag for LA rebalancer .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.9.x) [tools] --load imbalance threshold flag for LA rebalancer

2019-02-07 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12394 to review the following change. Change subject: [tools] --load_imbalance_threshold flag for LA rebalancer

[kudu-CR](branch-1.9.x) WIP: add release notes for 1.9.0

2019-02-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12389 ) Change subject: WIP: add release notes for 1.9.0 .. Patch Set 1: (1 comment)

[kudu-CR] [Java] Add PartialRow and RowResult getObject API

2019-02-07 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12393 Change subject: [Java] Add PartialRow and RowResult getObject API .. [Java] Add PartialRow and RowResult getObject API This patch adds

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 6: (17 comments) Just noticed a new patch set has

[kudu-CR] Rename DeadlineTracker to TimeoutTracker

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12373 ) Change subject: Rename DeadlineTracker to TimeoutTracker .. Patch Set 3: (1 comment)

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-02-07 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11797 to look at the new patch set (#7). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

[kudu-CR](branch-1.9.x) [rebalancer] location-aware rebalancer (part 10/n)

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12395 ) Change subject: [rebalancer] location-aware rebalancer (part 10/n) .. [rebalancer] location-aware rebalancer (part 10/n) This

[kudu-CR](branch-1.9.x) [rebalancer] location-aware rebalancer (part 10/n)

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

[kudu-CR](branch-1.9.x) [rebalancer] location-aware rebalancer (part 10/n)

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12395 ) Change subject: [rebalancer] location-aware rebalancer (part 10/n) .. Patch Set 1: Verified+1 unrelated flake in

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

2019-02-07 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 5: (1 comment) I temporarily added the snapshot

[kudu-CR] KUDU-1868: Part 1: Add timer-based RPC timeouts

2019-02-07 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12338 to look at the new patch set (#7). Change subject: KUDU-1868: Part 1: Add timer-based RPC timeouts

[kudu-CR] KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

2019-02-07 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12363 to look at the new patch set (#4). Change subject: KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

[kudu-CR] [tools] --load imbalance threshold flag for LA rebalancer

2019-02-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12366 ) Change subject: [tools] --load_imbalance_threshold flag for LA rebalancer .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

2019-02-07 Thread Grant Henke (Code Review)
Grant Henke has uploaded a new patch set (#5) to the change originally created by Brian McDevitt. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. KUDU-2411:

[kudu-CR](branch-1.9.x) [tools] --load imbalance threshold flag for LA rebalancer

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12394 ) Change subject: [tools] --load_imbalance_threshold flag for LA rebalancer .. [tools] --load_imbalance_threshold flag for LA

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-02-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-1868: Part 1: Add timer-based RPC timeouts

2019-02-07 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12338 ) Change subject: KUDU-1868: Part 1: Add timer-based RPC timeouts .. Patch Set 5: (2 comments)

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-02-07 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11797 to look at the new patch set (#6). Change subject: [sentry] Integrate AuthzProvider into CatalogManager

[kudu-CR](branch-1.9.x) [rebalancer] location-aware rebalancer (part 10/n)

2019-02-07 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12395 to review the following change. Change subject: [rebalancer] location-aware rebalancer (part 10/n) ..

[kudu-CR] KUDU-2543 pt 1: basic checks for authz tokens

2019-02-07 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11751 ) Change subject: KUDU-2543 pt 1: basic checks for authz tokens .. KUDU-2543 pt 1: basic checks for authz tokens In preparation

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

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11892 ) Change subject: [rebalancer] location-aware rebalancer (part 10/n) .. [rebalancer] location-aware rebalancer (part 10/n) This

[kudu-CR] [tools] --load imbalance threshold flag for LA rebalancer

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12366 ) Change subject: [tools] --load_imbalance_threshold flag for LA rebalancer .. [tools] --load_imbalance_threshold flag for LA

[kudu-CR] KUDU-2411: Example integration test using KuduMiniCluster

2019-02-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12318 ) Change subject: KUDU-2411: Example integration test using KuduMiniCluster .. Patch Set 5: (1 comment)

[kudu-CR] [clock] fix VLOG() misprint in MockNtp

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12404 Change subject: [clock] fix VLOG() misprint in MockNtp .. [clock] fix VLOG() misprint in MockNtp Moved misplaced VLOG() into

[kudu-CR] [sentry] Integrate AuthzProvider into CatalogManager

2019-02-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11797 ) Change subject: [sentry] Integrate AuthzProvider into CatalogManager .. Patch Set 7: (22 comments)

[kudu-CR] KUDU-2674: [java] Add a Java KuduPartitioner API

2019-02-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12275 ) Change subject: KUDU-2674: [java] Add a Java KuduPartitioner API .. Patch Set 4: (4 comments)

[kudu-CR] KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

2019-02-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12363 ) Change subject: KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation .. Patch Set 2: (5 comments)

[kudu-CR] KUDU-2676 Support restoring tables over the maximum allowed replicas

2019-02-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12372 ) Change subject: KUDU-2676 Support restoring tables over the maximum allowed replicas .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [tools] --load imbalance threshold flag for LA rebalancer

2019-02-07 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/12366 to look at the new patch set (#3). Change subject: [tools] --load_imbalance_threshold flag for LA rebalancer

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

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