[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12406 Change subject: KUDU-2411: Mark the KuduTestHarness as public .. KUDU-2411: Mark the KuduTestHarness as public This patch marks the

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Brian McDevitt (Code Review)
Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12406 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR] [spark-tools] DistributedDataGenerator repartition support

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12411 Change subject: [spark-tools] DistributedDataGenerator repartition support .. [spark-tools] DistributedDataGenerator repartition support

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

2019-02-08 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12404 to look at the new patch set (#2). Change subject: [clock] fix VLOG() misprint in MockNtp .. [clock] fix VLOG()

[kudu-CR](branch-1.9.x) KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12408 Change subject: KUDU-2411: Mark the KuduTestHarness as public .. KUDU-2411: Mark the KuduTestHarness as public This patch marks the

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12406 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [spark] Factor out the Spark/Kudu row converter

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12410 Change subject: [spark] Factor out the Spark/Kudu row converter .. [spark] Factor out the Spark/Kudu row converter This patch factors

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

2019-02-08 Thread Grant Henke (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12393 to look at the new patch set (#2). Change subject: [Java] Add PartialRow and RowResult getObject API .. [Java] Add

[kudu-CR] [Java] Add setRow to Operation

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12409 Change subject: [Java] Add setRow to Operation .. [Java] Add setRow to Operation Adds a setRow method to Operation to allow for existing

[kudu-CR](branch-1.9.x) KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: KUDU-2411: Mark the KuduTestHarness as public .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12408 To unsubscribe,

[kudu-CR](branch-1.9.x) KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12408 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12406 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. KUDU-2411: Mark the KuduTestHarness as public This patch marks

[kudu-CR](branch-1.9.x) KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12408 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. KUDU-2411: Mark the KuduTestHarness as public This patch marks

[kudu-CR](branch-1.9.x) KUDU-2411: Mark the KuduTestHarness as public

2019-02-08 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12408 ) Change subject: KUDU-2411: Mark the KuduTestHarness as public .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2543 pt 3 java: pass around authz tokens

2019-02-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12279 ) Change subject: KUDU-2543 pt 3 java: pass around authz tokens .. Patch Set 1: (4 comments)

[kudu-CR] Rename DeadlineTracker to TimeoutTracker

2019-02-08 Thread Will Berkeley (Code Review)
Will Berkeley 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] KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation

2019-02-08 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 4: Verified+1 -- To view, visit

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

2019-02-08 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: KUDU-1868 Part 2: Eliminate socket read timeout except for negotiation .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] Rename DeadlineTracker to TimeoutTracker

2019-02-08 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: Code-Review+2 (1 comment)

[kudu-CR] [tools] Add tool to list data dirs for a tablet

2019-02-08 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12417 Change subject: [tools] Add tool to list data dirs for a tablet .. [tools] Add tool to list data dirs for a tablet It's hard to find

[kudu-CR] KUDU-2670: split more scanner and add concurrent

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12323 ) Change subject: KUDU-2670: split more scanner and add concurrent .. Patch Set 2: (15 comments) Thank you for this contribution. I did

[kudu-CR] Schema copy and move constructors/operators must copy max col id

2019-02-08 Thread Mike Percy (Code Review)
Hello Will Berkeley, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12415 to review the following change. Change subject: Schema copy and move constructors/operators must copy max_col_id_

[kudu-CR] KUDU-2645: schema: Validate that defaults are provided for virtual columns

2019-02-08 Thread Mike Percy (Code Review)
Hello Will Berkeley, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12416 to review the following change. Change subject: KUDU-2645: schema: Validate that defaults are provided for virtual columns

[kudu-CR] [tools] Add tool to list data dirs for a tablet

2019-02-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12417 ) Change subject: [tools] Add tool to list data dirs for a tablet .. Patch Set 1: (1 comment)

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

2019-02-08 Thread Todd Lipcon (Code Review)
Todd Lipcon 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: (14 comments)

[kudu-CR] KUDU-2672: Spark write to kudu, too many machines write to one tserver.

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12341 ) Change subject: KUDU-2672: Spark write to kudu, too many machines write to one tserver. .. Patch Set 1: Checking in. Are you

[kudu-CR] KUDU-2645: schema: Add support for virtual columns in projections

2019-02-08 Thread Mike Percy (Code Review)
Hello Will Berkeley, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12204 to look at the new patch set (#2). Change subject: KUDU-2645: schema: Add support for virtual columns in projections

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

2019-02-08 Thread Alexey Serbin (Code Review)
Alexey Serbin 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: (7 comments)

[kudu-CR] Schema copy and move constructors/operators must copy max col id

2019-02-08 Thread Mike Percy (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/12415 to look at the new patch set (#2). Change subject: Schema copy and move constructors/operators must copy max_col_id_

[kudu-CR] KUDU-2645: schema: Validate that defaults are provided for virtual columns

2019-02-08 Thread Mike Percy (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/12416 to look at the new patch set (#3). Change subject: KUDU-2645: schema: Validate that defaults are provided for virtual columns

[kudu-CR] KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

2019-02-08 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12205 to look at the new patch set (#2). Change subject: KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

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

2019-02-08 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12389 to look at the new patch set (#2). Change subject: WIP: add release notes for 1.9.0

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

2019-02-08 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 2: (21 comments)

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

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke 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: (2 comments)

[kudu-CR] [Java] Add setRow to Operation

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12409 ) Change subject: [Java] Add setRow to Operation .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/12409 To

[kudu-CR] [Java] Add setRow to Operation

2019-02-08 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [Java] Add setRow to Operation .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12409 To unsubscribe, visit

[kudu-CR] KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator

2019-02-08 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12205 to look at the new patch set (#3). Change subject: KUDU-2645: tablet: Support deduplication of deleted rows in MergeIterator