[kudu-CR] util: fix logged Kudu URL

Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9280 to look at the new patch set (#3). Change subject: util: fix logged Kudu URL .. util: fix logged

[kudu-CR] docs: update docs for update dirs tool

Hello Alex Rodoni, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9110 to look at the new patch set (#3). Change subject: docs: update docs for update_dirs tool

[kudu-CR] docs: update docs for update dirs tool

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9110 ) Change subject: docs: update docs for update_dirs tool .. Patch Set 3: (2 comments)

[kudu-CR] [consensus peers] micro-optimization on controller status

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9387 ) Change subject: [consensus_peers] micro-optimization on controller status .. Patch Set 2: sure, seems fine, but for 0.03% of the CPU in

[kudu-CR] [consensus peers] micro-optimization on controller status

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9387 ) Change subject: [consensus_peers] micro-optimization on controller status .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2305: Fix local variable usage to handle 2GB messages

Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/9355 ) Change subject: KUDU-2305: Fix local variable usage to handle 2GB messages .. Patch Set 3: Ping. Impala also needs this patch once this

[kudu-CR] [consensus peers] micro-optimization on controller status

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9387 ) Change subject: [consensus_peers] micro-optimization on controller status .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] docs: add warning to wait between FS rebuilds

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9279 to look at the new patch set (#2). Change subject: docs: add warning to wait between FS rebuilds .. docs: add

[kudu-CR] tools: add more detail about unsafe change config

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9278 to look at the new patch set (#2). Change subject: tools: add more detail about unsafe_change_config .. tools: add

[kudu-CR] fs: clarify error message at startup

Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9281 to look at the new patch set (#2). Change subject: fs: clarify error message at startup .. fs: clarify

[kudu-CR] util: fix logged Kudu URL

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9280 to look at the new patch set (#2). Change subject: util: fix logged Kudu URL .. util: fix logged Kudu URL

[kudu-CR] KUDU-2039: /masters can show the wrong list of masters

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9378 ) Change subject: KUDU-2039: /masters can show the wrong list of masters .. Patch Set 2: Code-Review+1 (2 comments)

[kudu-CR] [java] fix the regression test for KUDU-2267/KUDU-2319

Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9388 Change subject: [java] fix the regression test for KUDU-2267/KUDU-2319 .. [java] fix the regression test for KUDU-2267/KUDU-2319 Previously,

[kudu-CR] KUDU-2297 (part 4): periodically dump stacks to diagnostics log

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9330 ) Change subject: KUDU-2297 (part 4): periodically dump stacks to diagnostics log .. Patch Set 8: Code-Review+2 (2 comments)

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

Hello Alex Rodoni, Mike Percy, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8823 to look at the new patch set (#12). Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

[kudu-CR] [java] add test to reproduce KUDU-2267

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9071 ) Change subject: [java] add test to reproduce KUDU-2267 .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-2297 (part 1): refactor metrics log out of ServerBase

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9326 ) Change subject: KUDU-2297 (part 1): refactor metrics log out of ServerBase .. Patch Set 7: (2 comments)

[kudu-CR] KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing

Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9319 ) Change subject: KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing .. KUDU-2291 (part 6) stacks: use libunwind

[kudu-CR] KUDU-2291 (part 5): allow collecting stack traces asynchronously

Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9318 ) Change subject: KUDU-2291 (part 5): allow collecting stack traces asynchronously .. KUDU-2291 (part 5): allow collecting stack

[kudu-CR] rowset metadata: cache min/max encoded keys

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9372 ) Change subject: rowset_metadata: cache min/max encoded keys .. Patch Set 2: (1 comment)

[kudu-CR] rowset metadata: cache min/max encoded keys

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9372 ) Change subject: rowset_metadata: cache min/max encoded keys .. Patch Set 2: before we spent too much time with this path I think

[kudu-CR] [consensus peers] micro-optimization on controller status

Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9387 to look at the new patch set (#2). Change subject: [consensus_peers] micro-optimization on controller status

[kudu-CR] [consensus peers] micro-optimization on controller status

Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9387 Change subject: [consensus_peers] micro-optimization on controller status .. [consensus_peers] micro-optimization on controller status

[kudu-CR] KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing

Hello Will Berkeley, Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9319 to look at the new patch set (#8). Change subject: KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing

[kudu-CR] java: improve error messages when tokens are not used

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9070 ) Change subject: java: improve error messages when tokens are not used .. Patch Set 1: > Would be great to get this in for 1.7. Overall, the

[kudu-CR] [Spark] Simplifiy creating range partitioned tables

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9379 ) Change subject: [Spark] Simplifiy creating range partitioned tables .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR] rowset metadata: cache min/max encoded keys

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9372 to look at the new patch set (#2). Change subject: rowset_metadata: cache min/max encoded keys .. rowset_metadata:

[kudu-CR] KUDU-2039: /masters can show the wrong list of masters

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9378 ) Change subject: KUDU-2039: /masters can show the wrong list of masters .. Patch Set 2: Code-Review+1 (3 comments) Looks good! Just

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

Hello Alex Rodoni, Mike Percy, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8823 to look at the new patch set (#11). Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 10: (17 comments)

[kudu-CR] docs: improvements to transaction semantics

Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9235 ) Change subject: docs: improvements to transaction semantics .. docs: improvements to transaction semantics Change-Id:

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9363 ) Change subject: KUDU-721: Support range partitions on decimal columns .. KUDU-721: Support range partitions on decimal columns

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 10: (17 comments)

[kudu-CR] KUDU-2297 (part 2). Convert diagnostics log to a format closer to glog

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9327 ) Change subject: KUDU-2297 (part 2). Convert diagnostics log to a format closer to glog .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2297 (part 1): refactor metrics log out of ServerBase

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9326 ) Change subject: KUDU-2297 (part 1): refactor metrics log out of ServerBase .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-2297 (part 1): refactor metrics log out of ServerBase

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9326 ) Change subject: KUDU-2297 (part 1): refactor metrics log out of ServerBase .. Patch Set 7: (1 comment) nice clean lifecycle

[kudu-CR] KUDU-721: [Flume] Add DECIMAL type support

Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9365 ) Change subject: KUDU-721: [Flume] Add DECIMAL type support .. KUDU-721: [Flume] Add DECIMAL type support Adds decimal column

[kudu-CR] KUDU-721: [Flume] Add DECIMAL type support

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9365 ) Change subject: KUDU-721: [Flume] Add DECIMAL type support .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] KUDU-721: [Flume] Add DECIMAL type support

Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9365 ) Change subject: KUDU-721: [Flume] Add DECIMAL type support .. Patch Set 2: (1 comment)

[kudu-CR] [itests] fix flake in raft consensus nonvoter-itest

Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9380 ) Change subject: [itests] fix flake in raft_consensus_nonvoter-itest .. [itests] fix flake in raft_consensus_nonvoter-itest

[kudu-CR] [itests] fix flake in raft consensus nonvoter-itest

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9380 ) Change subject: [itests] fix flake in raft_consensus_nonvoter-itest .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-721: [Flume] Add DECIMAL type support

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9365 ) Change subject: KUDU-721: [Flume] Add DECIMAL type support .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2291 (part 5): allow collecting stack traces asynchronously

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9318 ) Change subject: KUDU-2291 (part 5): allow collecting stack traces asynchronously .. Patch Set 7: Code-Review+2 (1 comment)

[kudu-CR] docs: improvements to transaction semantics

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9235 ) Change subject: docs: improvements to transaction semantics .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2297 (part 3): refactor process-wide stack collection out of /stacks

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9329 ) Change subject: KUDU-2297 (part 3): refactor process-wide stack collection out of /stacks .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Hello Alexey Serbin, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9363 to look at the new patch set (#4). Change subject: KUDU-721: Support range partitions on decimal columns

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9363 ) Change subject: KUDU-721: Support range partitions on decimal columns .. Patch Set 3: (4 comments)

[kudu-CR] docs: improvements to transaction semantics

Hello Alex Rodoni, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9235 to look at the new patch set (#3). Change subject: docs: improvements to transaction semantics

[kudu-CR] KUDU-2259: add real user to AuthenticationCredentialsPB

Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9374 to look at the new patch set (#2). Change subject: KUDU-2259: add real user to AuthenticationCredentialsPB

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9363 ) Change subject: KUDU-721: Support range partitions on decimal columns .. Patch Set 3: (1 comment)

[kudu-CR] WIP [tests] scenario to repro off-by-one error in TestWorkload

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9277 ) Change subject: WIP [tests] scenario to repro off-by-one error in TestWorkload .. Patch Set 1: > Patch Set 1: > > I briefly tried this

[kudu-CR] KUDU-2291 (part 5): allow collecting stack traces asynchronously

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9318 ) Change subject: KUDU-2291 (part 5): allow collecting stack traces asynchronously .. Patch Set 5: (2 comments)

[kudu-CR] KUDU-2291 (part 5): allow collecting stack traces asynchronously

Hello Will Berkeley, Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9318 to look at the new patch set (#7). Change subject: KUDU-2291 (part 5): allow collecting stack traces asynchronously

[kudu-CR] Fix move constructors to be noexcept

Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9382 to review the following change. Change subject: Fix move constructors to be noexcept .. Fix move constructors to be

[kudu-CR] KUDU-2297 (part 1): refactor metrics log out of ServerBase

Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9326 ) Change subject: KUDU-2297 (part 1): refactor metrics log out of ServerBase .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [java] add test to reproduce KUDU-2267

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9071 ) Change subject: [java] add test to reproduce KUDU-2267 .. Patch Set 5: (1 comment)

[kudu-CR] docs: improvements to transaction semantics

Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9235 ) Change subject: docs: improvements to transaction semantics .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-1704: add READ YOUR WRITES scan mode

Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_YOUR_WRITES scan mode .. KUDU-1704: add READ_YOUR_WRITES scan mode This patch adds a new read

[kudu-CR] KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9319 ) Change subject: KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing .. Patch Set 8: Code-Review+2 (1 comment)

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9363 ) Change subject: KUDU-721: Support range partitions on decimal columns .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1704: add READ YOUR WRITES scan mode

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_YOUR_WRITES scan mode .. Patch Set 18: Code-Review+1 (1 comment)

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9363 ) Change subject: KUDU-721: Support range partitions on decimal columns .. Patch Set 3: (2 comments)

[kudu-CR] KUDU-2297 (part 4): periodically dump stacks to diagnostics log

Hello Will Berkeley, Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9330 to look at the new patch set (#8). Change subject: KUDU-2297 (part 4): periodically dump stacks to diagnostics log

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Hello Alexey Serbin, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9363 to look at the new patch set (#3). Change subject: KUDU-721: Support range partitions on decimal columns

[kudu-CR] KUDU-721: Support range partitions on decimal columns

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9363 ) Change subject: KUDU-721: Support range partitions on decimal columns .. Patch Set 2: Code-Review+1 (5 comments)

[kudu-CR] docs: improvements to transaction semantics

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9235 ) Change subject: docs: improvements to transaction semantics .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1704: add READ YOUR WRITES scan mode

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_YOUR_WRITES scan mode .. Patch Set 18: Code-Review+2 -- To view, visit

[kudu-CR] [itests] fix flake in raft consensus nonvoter-itest

Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9380 ) Change subject: [itests] fix flake in raft_consensus_nonvoter-itest .. Patch Set 2: (1 comment)

[kudu-CR] [itests] fix flake in raft consensus nonvoter-itest

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9380 to look at the new patch set (#2). Change subject: [itests] fix flake in raft_consensus_nonvoter-itest .. [itests]

[kudu-CR] [itests] fix flake in raft consensus nonvoter-itest

Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9380 Change subject: [itests] fix flake in raft_consensus_nonvoter-itest .. [itests] fix flake in raft_consensus_nonvoter-itest Fixed flake

[kudu-CR] [Spark] Simplifiy creating range partitioned tables

Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9379 Change subject: [Spark] Simplifiy creating range partitioned tables .. [Spark] Simplifiy creating range partitioned tables In order to

[kudu-CR] KUDU-2039: /masters can show the wrong list of masters

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9378 to look at the new patch set (#2). Change subject: KUDU-2039: /masters can show the wrong list of masters ..

[kudu-CR] KUDU-2039: /masters can show the wrong list of masters

Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9378 Change subject: KUDU-2039: /masters can show the wrong list of masters .. KUDU-2039: /masters can show the wrong list of masters The

[kudu-CR] KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9319 ) Change subject: KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9319 ) Change subject: KUDU-2291 (part 6) stacks: use libunwind directly for stack tracing .. Patch Set 7: (1 comment)

[kudu-CR] rw semaphore: dont include debug-util.h when not necessary

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9328 ) Change subject: rw_semaphore: dont include debug-util.h when not necessary .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-1704: add READ YOUR WRITES scan mode

Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: KUDU-1704: add READ_YOUR_WRITES scan mode .. Patch Set 18: Code-Review+1 -- To view, visit