[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12180 ) Change subject: Fix DOS line endings in TestServerInfo.java .. Patch Set 4: Verified+1 Unrelated flake in raft_consensus-itest. --

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 1: (1 comment)

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12206 to look at the new patch set (#2). Change subject: [examples] fix name of the class for spark-submit

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 2: (1 comment)

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR] WIP: KUDU-2645. tablet: Add ghost row de-duplication support to MergeIterator

2019-01-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12205 ) Change subject: WIP: KUDU-2645. tablet: Add ghost row de-duplication support to MergeIterator .. Patch Set 1: (1 comment)

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12175 to look at the new patch set (#7). Change subject: Support location awareness in READ_CLOSEST for the Java client

[kudu-CR] KUDU-2640: Add Spark Structured Streaming Sink

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: KUDU-2640: Add Spark Structured Streaming Sink .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12087 To unsubscribe,

[kudu-CR] WIP: tablet: Add support for virtual columns to Schema

2019-01-09 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12204 Change subject: WIP: tablet: Add support for virtual columns to Schema .. WIP: tablet: Add support for virtual columns to Schema Needs

[kudu-CR] generic iterators: short-circuit MergeIterState::PullNextBlock

2019-01-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/12157 ) Change subject: generic_iterators: short-circuit MergeIterState::PullNextBlock .. Patch Set 6: (1 comment)

[kudu-CR] generic iterators: short-circuit MergeIterState::PullNextBlock

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12157 to look at the new patch set (#7). Change subject: generic_iterators: short-circuit MergeIterState::PullNextBlock

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 2: (1 comment)

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12174 ) Change subject: Assign locations to tablet servers and the client in Java .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12180 ) Change subject: Fix DOS line endings in TestServerInfo.java .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2640: Add Spark Structured Streaming Sink

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12087 ) Change subject: KUDU-2640: Add Spark Structured Streaming Sink .. Patch Set 4: Verified+1 Unrelated test failure. -- To view, visit

[kudu-CR] Reduce election-related logging

2019-01-09 Thread Will Berkeley (Code Review)
Hello Mike Percy, Kudu Jenkins, Andrew Wong, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12200 to look at the new patch set (#2). Change subject: Reduce election-related logging

[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: Fix DOS line endings in TestServerInfo.java .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/12180 To unsubscribe,

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12194 to look at the new patch set (#3). Change subject: Eliminate redundant VLOG_IS_ON calls

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12206 Change subject: [examples] fix name of the class for spark-submit .. [examples] fix name of the class for spark-submit I also removed

[kudu-CR] [examples] a small update on SparkExample

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12208 Change subject: [examples] a small update on SparkExample .. [examples] a small update on SparkExample Updated the Spark example: *

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 2: (2 comments)

[kudu-CR] WIP: KUDU-2645. tablet: Add ghost row de-duplication support to MergeIterator

2019-01-09 Thread Mike Percy (Code Review)
Mike Percy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12205 Change subject: WIP: KUDU-2645. tablet: Add ghost row de-duplication support to MergeIterator .. WIP: KUDU-2645. tablet: Add ghost row

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Patch Set 5: (2 comments)

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 12: (6 comments)

[kudu-CR] generic iterators: basic MergeIterator dominance

2019-01-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12197 ) Change subject: generic_iterators: basic MergeIterator dominance .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2640: Add Spark Structured Streaming Sink

2019-01-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12087 ) Change subject: KUDU-2640: Add Spark Structured Streaming Sink .. Patch Set 4: (2 comments)

[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12180 ) Change subject: Fix DOS line endings in TestServerInfo.java .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12174 ) Change subject: Assign locations to tablet servers and the client in Java .. Assign locations to tablet servers and the client

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Eliminate redundant VLOG_IS_ON calls In an expression like VLOG(3)

[kudu-CR] [examples] a small update on SparkExample

2019-01-09 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12208 to look at the new patch set (#2). Change subject: [examples] a small update on SparkExample .. [examples] a small

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [examples] a small update on SparkExample

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12208 ) Change subject: [examples] a small update on SparkExample .. Patch Set 2: (2 comments)

[kudu-CR] WIP: KUDU-2645. tablet: Add ghost row de-duplication support to MergeIterator

2019-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12205 ) Change subject: WIP: KUDU-2645. tablet: Add ghost row de-duplication support to MergeIterator .. Patch Set 1: (2 comments) Just

[kudu-CR] Reduce election-related logging

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12200 ) Change subject: Reduce election-related logging .. Patch Set 2: (2 comments)

[kudu-CR] Reduce election-related logging

2019-01-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12200 ) Change subject: Reduce election-related logging .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12200

[kudu-CR] [examples] fix name of the class for spark-submit

2019-01-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/12206 ) Change subject: [examples] fix name of the class for spark-submit .. Patch Set 1: (1 comment)

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Will Berkeley (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12194 to look at the new patch set (#4). Change subject: Eliminate redundant VLOG_IS_ON calls

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Support location awareness in READ_CLOSEST

[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12180 ) Change subject: Fix DOS line endings in TestServerInfo.java .. Fix DOS line endings in TestServerInfo.java Change-Id:

[kudu-CR] [examples] a small update on SparkExample

2019-01-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12208 ) Change subject: [examples] a small update on SparkExample .. Patch Set 2: (3 comments)

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#13). Change subject: KUDU-2636: LBM supports deleting dead and full containers

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 12: (6 comments)

[kudu-CR] [examples] a small update on SparkExample

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12208 ) Change subject: [examples] a small update on SparkExample .. Patch Set 2: (5 comments)

[kudu-CR] [examples] a small update on SparkExample

2019-01-09 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, Andrew Wong, Mitch Barnett, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12208 to look at the new patch set (#3). Change subject: [examples] a small update on SparkExample

[kudu-CR] KUDU-2656: pass IOContext to ValidateDeltaOrder

2019-01-09 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12178 ) Change subject: KUDU-2656: pass IOContext to ValidateDeltaOrder .. KUDU-2656: pass IOContext to ValidateDeltaOrder Previously,

[kudu-CR] Reduce severity of "Error trying to read ahead of the log" log message

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: Reduce severity of "Error trying to read ahead of the log" log message .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] Support location awareness in READ CLOSEST for the C++ client

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12138 ) Change subject: Support location awareness in READ_CLOSEST for the C++ client .. Patch Set 8: A follow-up's precommit uncovered

[kudu-CR] Reduce severity of "Error trying to read ahead of the log" log message

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12185 ) Change subject: Reduce severity of "Error trying to read ahead of the log" log message .. Reduce severity of "Error trying to

[kudu-CR] Reduce severity of "Error trying to read ahead of the log" log message

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12185 ) Change subject: Reduce severity of "Error trying to read ahead of the log" log message .. Patch Set 2: Verified+1 Saw what looks

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 11: All right, calling 'SyncDir()' has been removed.

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#12). Change subject: KUDU-2636: LBM supports deleting dead and full containers

[kudu-CR] Support location awareness in READ CLOSEST for the C++ client

2019-01-09 Thread Will Berkeley (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12138 to look at the new patch set (#9). Change subject: Support location awareness in READ_CLOSEST for the C++ client

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12174 ) Change subject: Assign locations to tablet servers and the client in Java .. Patch Set 4: (1 comment)

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Patch Set 3: (1 comment)

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12175 to look at the new patch set (#4). Change subject: Support location awareness in READ_CLOSEST for the Java client

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12174 to look at the new patch set (#5). Change subject: Assign locations to tablet servers and the client in Java

[kudu-CR] generic iterators: short-circuit MergeIterState::PullNextBlock

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12157 to look at the new patch set (#4). Change subject: generic_iterators: short-circuit MergeIterState::PullNextBlock

[kudu-CR] generic iterators: move MergeIterState into the header

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12195 to review the following change. Change subject: generic_iterators: move MergeIterState into the header

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Patch Set 4: Code-Review+1 -- To view, visit

[kudu-CR] Support location awareness in READ CLOSEST for the C++ client

2019-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12138 ) Change subject: Support location awareness in READ_CLOSEST for the C++ client .. Patch Set 9: Code-Review+1 -- To view, visit

[kudu-CR] generic iterators: basic MergeIterator dominance

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12197 to review the following change. Change subject: generic_iterators: basic MergeIterator dominance

[kudu-CR] generic iterators: prep for MergeIterator dominance

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Grant Henke, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/12196 to review the following change. Change subject: generic_iterators: prep for MergeIterator dominance

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 1: (2 comments)

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12174 ) Change subject: Assign locations to tablet servers and the client in Java .. Patch Set 5: Code-Review+1 (1 comment)

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12194 Change subject: Eliminate redundant VLOG_IS_ON calls .. Eliminate redundant VLOG_IS_ON calls In an expression like VLOG(3) <<

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12174 ) Change subject: Assign locations to tablet servers and the client in Java .. Patch Set 5: (1 comment)

[kudu-CR] Support location awareness in READ CLOSEST for the Java client

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12175 ) Change subject: Support location awareness in READ_CLOSEST for the Java client .. Patch Set 4: (3 comments)

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 1: (1 comment)

[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12180 ) Change subject: Fix DOS line endings in TestServerInfo.java .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] Fix DOS line endings in TestServerInfo.java

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12180 ) Change subject: Fix DOS line endings in TestServerInfo.java .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] Support location awareness in READ CLOSEST for the C++ client

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12138 ) Change subject: Support location awareness in READ_CLOSEST for the C++ client .. Patch Set 9: Code-Review+1 (1 comment)

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2195. Add additional gflag to force sync of consensus metadata

2019-01-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12186 ) Change subject: KUDU-2195. Add additional gflag to force sync of consensus metadata .. KUDU-2195. Add additional gflag to force

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 12: (1 comment)

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 1: (1 comment)

[kudu-CR] Eliminate redundant VLOG IS ON calls

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12194 ) Change subject: Eliminate redundant VLOG_IS_ON calls .. Patch Set 1: (1 comment)

[kudu-CR] Support location awareness in READ CLOSEST for the C++ client

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12138 ) Change subject: Support location awareness in READ_CLOSEST for the C++ client .. Patch Set 9: Code-Review+2 Nice catch regarding the

[kudu-CR](gh-pages) [build] Update repo URL

2019-01-09 Thread Attila Bukor (Code Review)
Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12151 ) Change subject: [build] Update repo URL .. [build] Update repo URL According to Apache Infra Team the git repo needs to be

[kudu-CR] [build] Update repo URL

2019-01-09 Thread Attila Bukor (Code Review)
Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12150 ) Change subject: [build] Update repo URL .. [build] Update repo URL According to Apache Infra Team the git repo needs to be

[kudu-CR](gh-pages) [build] Update repo URL

2019-01-09 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/12151 ) Change subject: [build] Update repo URL .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/12151 To

[kudu-CR] generic iterators: short-circuit MergeIterState::PullNextBlock

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12157 to look at the new patch set (#5). Change subject: generic_iterators: short-circuit MergeIterState::PullNextBlock

[kudu-CR](gh-pages) [build] Update repo URL

2019-01-09 Thread Attila Bukor (Code Review)
Attila Bukor has removed a vote on this change. Change subject: [build] Update repo URL .. Removed Verified-1 by Attila Bukor -- To view, visit http://gerrit.cloudera.org:8080/12151 To unsubscribe, visit

[kudu-CR] [build] Update repo URL

2019-01-09 Thread Attila Bukor (Code Review)
Attila Bukor has removed a vote on this change. Change subject: [build] Update repo URL .. Removed Verified-1 by Attila Bukor -- To view, visit http://gerrit.cloudera.org:8080/12150 To unsubscribe, visit

[kudu-CR] generic iterators: assorted cleanup

2019-01-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12156 ) Change subject: generic_iterators: assorted cleanup .. Patch Set 3: (4 comments)

[kudu-CR] generic iterators: short-circuit MergeIterState::PullNextBlock

2019-01-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12157 ) Change subject: generic_iterators: short-circuit MergeIterState::PullNextBlock .. Patch Set 4: (3 comments)

[kudu-CR] generic iterators: basic MergeIterator dominance

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12197 to look at the new patch set (#2). Change subject: generic_iterators: basic MergeIterator dominance

[kudu-CR] rpc: add TCP socket statistics to /rpcz

2019-01-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12184 ) Change subject: rpc: add TCP socket statistics to /rpcz .. Patch Set 1: (4 comments) LGTM and it compiles and runs on macOS. Just

[kudu-CR] KUDU-2640: Add Spark Structured Streaming Sink

2019-01-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12087 ) Change subject: KUDU-2640: Add Spark Structured Streaming Sink .. Patch Set 2: (4 comments)

[kudu-CR] KUDU-2640: Add Spark Structured Streaming Sink

2019-01-09 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12087 to look at the new patch set (#3). Change subject: KUDU-2640: Add Spark Structured Streaming Sink

[kudu-CR] KUDU-2640: Add Spark Structured Streaming Sink

2019-01-09 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Andrew Wong, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12087 to look at the new patch set (#4). Change subject: KUDU-2640: Add Spark Structured Streaming Sink

[kudu-CR] generic iterators: move MergeIterState into the header

2019-01-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12195 ) Change subject: generic_iterators: move MergeIterState into the header .. Patch Set 2: (1 comment)

[kudu-CR] rpc: add TCP socket statistics to /rpcz

2019-01-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12184 ) Change subject: rpc: add TCP socket statistics to /rpcz .. Patch Set 1: (1 comment)

[kudu-CR] generic iterators: prep for MergeIterator dominance

2019-01-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12196 ) Change subject: generic_iterators: prep for MergeIterator dominance .. Patch Set 2: (2 comments)

[kudu-CR] Reduce election-related logging

2019-01-09 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12200 Change subject: Reduce election-related logging .. Reduce election-related logging Frequent simultaneous pre-elections on lots of

[kudu-CR] generic iterators: basic MergeIterator dominance

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12197 to look at the new patch set (#3). Change subject: generic_iterators: basic MergeIterator dominance

[kudu-CR] generic iterators: short-circuit MergeIterState::PullNextBlock

2019-01-09 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12157 to look at the new patch set (#6). Change subject: generic_iterators: short-circuit MergeIterState::PullNextBlock

[kudu-CR] Assign locations to tablet servers and the client in Java

2019-01-09 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12174 to look at the new patch set (#7). Change subject: Assign locations to tablet servers and the client in Java

  1   2   >