[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (5 comments)

[kudu-CR] wire protocol: optimize RewriteRowBlockPointers

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Andrew Wong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8554 to review the following change. Change subject: wire_protocol: optimize RewriteRowBlockPointers .. wire_protocol:

[kudu-CR] Add the ability to enable xray instrumentation

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8537 ) Change subject: Add the ability to enable xray instrumentation .. Add the ability to enable xray instrumentation Change-Id:

[kudu-CR] Add public INT128 column type support

2017-11-15 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8553 Change subject: Add public INT128 column type support .. Add public INT128 column type support This patch adds basic support to the c++

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8533 to look at the new patch set (#7). Change subject: Add initial internal INT128/__int128 support

[kudu-CR] docs: clarify at-rest encryption known limitation

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8556 to review the following change. Change subject: docs: clarify at-rest encryption known limitation .. docs:

[kudu-CR] KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8245 ) Change subject: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable .. Patch Set 6: r5 addresses

[kudu-CR] KUDU-2215. kernel stack watchdog: avoid blocking thread exit

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Andrew Wong, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8536 to look at the new patch set (#4). Change subject: KUDU-2215. kernel_stack_watchdog: avoid blocking thread exit

[kudu-CR] introduce closed mvcc and stopped tablets

2017-11-15 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/7439 ) Change subject: introduce closed mvcc and stopped tablets .. Patch Set 24: (3 comments) Fixed tidy warnings. This is ready for review.

[kudu-CR] introduce closed mvcc and stopped tablets

2017-11-15 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Mike Percy, David Ribeiro Alves, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7439 to look at the new patch set (#25). Change subject: introduce closed mvcc and stopped tablets

[kudu-CR] docs: clarify at-rest encryption known limitation

2017-11-15 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8556 ) Change subject: docs: clarify at-rest encryption known limitation .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] introduce closed mvcc and stopped tablets

2017-11-15 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/7439 ) Change subject: introduce closed mvcc and stopped tablets .. Patch Set 23: (22 comments)

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8533 to look at the new patch set (#8). Change subject: Add initial internal INT128/__int128 support

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 8: (5 comments)

[kudu-CR] KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Mike Percy, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8245 to look at the new patch set (#5). Change subject: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

[kudu-CR] KUDU-2215. kernel stack watchdog: avoid blocking thread exit

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Andrew Wong, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8536 to look at the new patch set (#3). Change subject: KUDU-2215. kernel_stack_watchdog: avoid blocking thread exit

[kudu-CR] ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8557 to review the following change. Change subject: ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8533 to look at the new patch set (#9). Change subject: Add initial internal INT128/__int128 support

[kudu-CR] KUDU-1454 [part 1]: update propagated timestamp on the driver

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8552 ) Change subject: KUDU-1454 [part 1]: update propagated timestamp on the driver .. Patch Set 1: (6 comments)

[kudu-CR] KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8245 ) Change subject: KUDU-2048. consensus: only evict unresponsive nodes if remaining voters are viable .. Patch Set 4: (8 comments)

[kudu-CR] introduce closed mvcc and stopped tablets

2017-11-15 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Mike Percy, David Ribeiro Alves, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7439 to look at the new patch set (#24). Change subject: introduce closed mvcc and stopped tablets

[kudu-CR] ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8557 to look at the new patch set (#2). Change subject: ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

[kudu-CR] docs: clarify at-rest encryption known limitation

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8556 ) Change subject: docs: clarify at-rest encryption known limitation .. docs: clarify at-rest encryption known limitation Some

[kudu-CR] [spark] Remove AsyncClient in KuduContext

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8551 ) Change subject: [spark] Remove AsyncClient in KuduContext .. Patch Set 1: (1 comment)

[kudu-CR] ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Tidy Bot, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8557 to look at the new patch set (#3). Change subject: ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

[kudu-CR] introduce closed mvcc and stopped tablets

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/7439 ) Change subject: introduce closed mvcc and stopped tablets .. Patch Set 25: (11 comments)

[kudu-CR] [RaftPeerPB] introduce replica health status

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8561 Change subject: [RaftPeerPB] introduce replica health status .. [RaftPeerPB] introduce replica health status Introduced replica health

[kudu-CR] [RaftPeerPB] introduce replica health status

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8561 ) Change subject: [RaftPeerPB] introduce replica health status .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-1078. Fix 'ops in future' error under load

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8563 ) Change subject: KUDU-1078. Fix 'ops in future' error under load .. Patch Set 1: Verified-1 seems this patch introduces some issue with

[kudu-CR] [RaftPeerPB] introduce new status fields

2017-11-15 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8561 to look at the new patch set (#2). Change subject: [RaftPeerPB] introduce new status fields

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-1454 [part 1]: update propagated timestamp on the driver

2017-11-15 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8552 ) Change subject: KUDU-1454 [part 1]: update propagated timestamp on the driver .. Patch Set 2: > Uploaded patch set 2: Patch Set 1 was

[kudu-CR] [spark] Remove AsyncClient in KuduContext

2017-11-15 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8551 ) Change subject: [spark] Remove AsyncClient in KuduContext .. Patch Set 2: > Uploaded patch set 2: Patch Set 1 was rebased. This is only a

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-1454 [part 2]: enable scan locality for Spark

2017-11-15 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8560 Change subject: KUDU-1454 [part 2]: enable scan locality for Spark .. KUDU-1454 [part 2]: enable scan locality for Spark This patch provides

[kudu-CR] [java] Add ReplicaSelection in KuduScanToken

2017-11-15 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8559 Change subject: [java] Add ReplicaSelection in KuduScanToken .. [java] Add ReplicaSelection in KuduScanToken This patch adds ReplicaSelection

[kudu-CR] WIP: tls socket: properly handle temporary socket errors in Writev

2017-11-15 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8570 ) Change subject: WIP: tls_socket: properly handle temporary socket errors in Writev .. Patch Set 1: Code-Review+1 The fix makes sense

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (1 comment)

[kudu-CR] dist test: enable sharding of a few more tests

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8565 to review the following change. Change subject: dist_test: enable sharding of a few more tests .. dist_test: enable

[kudu-CR] Mark hms client-test as RUN SERIAL to avoid timeouts

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8566 to review the following change. Change subject: Mark hms_client-test as RUN_SERIAL to avoid timeouts .. Mark

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (1 comment)

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (6 comments)

[kudu-CR] catalog manager tsk-itest: ensure that test eventually makes progress

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8567 to review the following change. Change subject: catalog_manager_tsk-itest: ensure that test eventually makes progress

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8533 to look at the new patch set (#10). Change subject: Add initial internal INT128/__int128 support

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 7: (1 comment)

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8533 ) Change subject: Add initial internal INT128/__int128 support .. Patch Set 11: (4 comments)

[kudu-CR] Add initial internal INT128/ int128 support

2017-11-15 Thread Grant Henke (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8533 to look at the new patch set (#11). Change subject: Add initial internal INT128/__int128 support

[kudu-CR] WIP: tls socket: properly handle temporary socket errors in Writev

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, Sailesh Mukil, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8570 to review the following change. Change subject: WIP: tls_socket: properly handle temporary socket errors in Writev

[kudu-CR] [RaftPeerPB] introduce new status fields

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8561 ) Change subject: [RaftPeerPB] introduce new status fields .. Patch Set 2: Just put up a gerrit for the TabletReplicaTest failures (which

[kudu-CR] Deflake tablet replica-test anchor and transaction count assertions

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8564 to review the following change. Change subject: Deflake tablet_replica-test anchor and transaction count assertions ..

[kudu-CR] KUDU-1078. Fix 'ops in future' error under load

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8563 ) Change subject: KUDU-1078. Fix 'ops in future' error under load .. Patch Set 2: actually decided that the test failures were unrelated

[kudu-CR] catalog manager tsk-itest: ensure that test eventually makes progress

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8567 ) Change subject: catalog_manager_tsk-itest: ensure that test eventually makes progress .. Patch Set 1: Code-Review+2 (2 comments)

[kudu-CR] ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Tidy Bot, Andrew Wong, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8557 to look at the new patch set (#4). Change subject: ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

[kudu-CR] ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8557 ) Change subject: ntp: dump NTP diagnostics when NTP is unsynchronized or error is too high .. Patch Set 4: Need to fiddle with the

[kudu-CR] KUDU-1078. Fix 'ops in future' error under load

2017-11-15 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8563 to review the following change. Change subject: KUDU-1078. Fix 'ops in future' error under load .. KUDU-1078. Fix 'ops

[kudu-CR] [RaftPeerPB] introduce new status fields

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8561 ) Change subject: [RaftPeerPB] introduce new status fields .. Patch Set 2: Verified+1 unrelated flakes in:

[kudu-CR] [RaftPeerPB] introduce new status fields

2017-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8561 ) Change subject: [RaftPeerPB] introduce new status fields .. Removed reviewer Kudu Jenkins with the following votes: *