[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++ cl

[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] 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) http://gerrit.cloudera.org:8080/#/c/8533/7/s

[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: opti

[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: I2a8d

[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 (#2). Change subject: KUDU-2215. kernel_stack_watchdog: avoid blocking thread exit .

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

2017-11-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8536 ) Change subject: KUDU-2215. kernel_stack_watchdog: avoid blocking thread exit .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:808

[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) http://

[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-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 comments

[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] 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: clar

[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 http://gerrit.c

[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)
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] 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) http://gerrit.cloudera.org:8080/#/c/7439/23//CO

[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) http://gerrit.cloudera.org:8080/#/c/8533/7/src

[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] 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] 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] 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] 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] 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] 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 users

[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) http://gerrit.cloudera.org:8080/#/c/8551/1//COMMIT

[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) http://gerrit.cloudera.org:80

[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) http://gerrit.cloudera.org:8080/#/c/8533/7/sr

[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) http://gerrit.cloudera.org:8080/#/c/8533/7/sr

[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] 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 s

[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: (2 comments) http://gerrit.cloudera.org:8080/#

[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 rebased

[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 re

[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] 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) http://gerrit.cloudera.org:8080/#/c/7439/23/src

[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 s

[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 slaves

[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) http://gerrit.cloudera.org:8080/#/c/8561/1/src/

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

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 replica health status .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8561/1/sr

[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)
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] 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 lo

[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: Parameters/TestRowSetTreeP

[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: * Verified-

[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] [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 I

[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) http://gerrit.cloudera.org:8080/#/c/8533/7/src/

[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 fl

[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 sh

[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 hms_clien

[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) http://gerrit.cloudera.org:8080/#/c/8533/7/sr

[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) http://gerrit.cloudera.org:8080/#/c/8533/7/src/

[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) http://gerrit.cloudera.org:8080/#/c/8533/10/

[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] 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) http://gerrit.cloudera.org:8080/#/c/8533/10/s

[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] 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) Than

[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] 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 t