[kudu-CR] WIP [client] expose non-voter replicas for kudu CLI tool

2017-11-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8586 ) Change subject: WIP [client] expose non-voter replicas for kudu CLI tool .. Patch Set 1: (4 comments) Looks alright to me until we can

[kudu-CR] KUDU-2218. tls socket: properly handle temporary socket errors in Writev

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8570 ) Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors in Writev .. Patch Set 3: > Change has been

[kudu-CR] KUDU-2218. tls socket: properly handle temporary socket errors in Writev

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8570 ) Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors in Writev .. Patch Set 2: Code-Review+2 (1 comment)

[kudu-CR] tablet: make various update paths atomic

2017-11-17 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Mike Percy, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8441 to look at the new patch set (#5). Change subject: tablet: make various update paths atomic

[kudu-CR] KUDU-2218. tls socket: properly handle temporary socket errors in Writev

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8570 ) Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors in Writev .. KUDU-2218. tls_socket: properly

[kudu-CR] tablet: make various update paths atomic

2017-11-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8441 ) Change subject: tablet: make various update paths atomic .. Patch Set 5: (9 comments)

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Anonymous Coward (Code Review)
Hello Chris George, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6010 to look at the new patch set (#2). Change subject: KUDU-1737 : Submit column characteristics via KuduContext

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

2017-11-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8565 ) Change subject: dist_test: enable sharding of a few more tests .. dist_test: enable sharding of a few more tests Adds sharding

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

2017-11-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8565 ) Change subject: dist_test: enable sharding of a few more tests .. Patch Set 2: Verified+1 Unrelated test flakiness issues w/ infra --

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

2017-11-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has removed a vote on this change. Change subject: dist_test: enable sharding of a few more tests .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/8565 To unsubscribe,

[kudu-CR] schema: micro-optimize num columns()

2017-11-17 Thread Todd Lipcon (Code Review)
Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8587 to review the following change. Change subject: schema: micro-optimize num_columns() .. schema: micro-optimize

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

2017-11-17 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 3: (1 comment)

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

2017-11-17 Thread Todd Lipcon (Code Review)
Todd Lipcon 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 2: (2 comments)

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

2017-11-17 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8567 to look at the new patch set (#2). Change subject: catalog_manager_tsk-itest: ensure that test eventually makes progress

[kudu-CR] WIP [catalog manager] introduce replica type matching policy

2017-11-17 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8161 to look at the new patch set (#5). Change subject: WIP [catalog manager] introduce replica type matching policy

[kudu-CR] wire protocol: optimize RewriteRowBlockPointers

2017-11-17 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/8554 to look at the new patch set (#2). Change subject: wire_protocol: optimize RewriteRowBlockPointers

[kudu-CR] WIP [client] expose non-voter replicas for kudu CLI tool

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8586 ) Change subject: WIP [client] expose non-voter replicas for kudu CLI tool .. Patch Set 2: (7 comments)

[kudu-CR] WIP [client] expose non-voter replicas for kudu CLI tool

2017-11-17 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8586 to look at the new patch set (#2). Change subject: WIP [client] expose non-voter replicas for kudu CLI tool

[kudu-CR] WIP [catalog manager] introduce replica type matching policy

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8161 ) Change subject: WIP [catalog manager] introduce replica type matching policy .. Patch Set 4: (2 comments)

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

2017-11-17 Thread Todd Lipcon (Code Review)
Todd Lipcon has removed a vote on this change. Change subject: KUDU-1078. Fix 'ops in future' error under load .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/8563 To unsubscribe,

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

2017-11-17 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 3: Verified+1 Unrelated test flakes -- To view, visit

[kudu-CR] [catalog manager] introduce replica type matching policy

2017-11-17 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8161 to look at the new patch set (#7). Change subject: [catalog manager] introduce replica type matching policy

[kudu-CR] tablet: make various update paths atomic

2017-11-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8441 ) Change subject: tablet: make various update paths atomic .. Patch Set 5: (1 comment)

[kudu-CR] schema: micro-optimize num columns()

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8587 ) Change subject: schema: micro-optimize num_columns() .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Anonymous Coward (Code Review)
saketa.chalamch...@gmail.com has posted comments on this change. ( http://gerrit.cloudera.org:8080/6010 ) Change subject: KUDU-1737 : Submit column characteristics via KuduContext .. Patch Set 3:

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/6010 ) Change subject: KUDU-1737 : Submit column characteristics via KuduContext .. Patch Set 3: >

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

2017-11-17 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 2: Code-Review+2 -- To view,

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

2017-11-17 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 3: (1 comment)

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Anonymous Coward (Code Review)
saketa.chalamch...@gmail.com has posted comments on this change. ( http://gerrit.cloudera.org:8080/6010 ) Change subject: KUDU-1737 : Submit column characteristics via KuduContext .. Patch Set 3: > (1 comment) Changed the

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Anonymous Coward (Code Review)
Hello Chris George, Dan Burkert, Kudu Jenkins, Hao Hao, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6010 to look at the new patch set (#3). Change subject: KUDU-1737 : Submit column characteristics via KuduContext

[kudu-CR] KUDU-1737 : Submit column characteristics via KuduContext

2017-11-17 Thread Anonymous Coward (Code Review)
saketa.chalamch...@gmail.com has posted comments on this change. ( http://gerrit.cloudera.org:8080/6010 ) Change subject: KUDU-1737 : Submit column characteristics via KuduContext .. Patch Set 3: > >

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

2017-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8567 ) Change subject: catalog_manager_tsk-itest: ensure that test eventually makes progress .. catalog_manager_tsk-itest: ensure

[kudu-CR] Bug: KUDU-1737

2017-11-17 Thread Anonymous Coward (Code Review)
saketa.chalamch...@gmail.com has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8591 Change subject: Bug: KUDU-1737 .. Bug: KUDU-1737 Submit defaultValue, desiredBlockSize, encoding and

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

2017-11-17 Thread Andrew Wong (Code Review)
Andrew Wong 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 4: Code-Review+2 -- To view, visit

[kudu-CR] [catalog manager] introduce replica type matching policy

2017-11-17 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8161 to look at the new patch set (#6). Change subject: [catalog manager] introduce replica type matching policy