[kudu-CR] Move configuration keys into its own class and make public

2018-01-11 Thread Clemens Valiente (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8920 to look at the new patch set (#2). Change subject: Move configuration keys into its own class and make public .

[kudu-CR] Move configuration keys into its own class and make public

2018-01-11 Thread Clemens Valiente (Code Review)
Clemens Valiente has posted comments on this change. ( http://gerrit.cloudera.org:8080/8920 ) Change subject: Move configuration keys into its own class and make public .. Patch Set 2: Todd, David, as promised I fixed the merg

[kudu-CR] KUDU-2256: Add GetTransferSize() to RpcContext

2018-01-11 Thread Lars Volker (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8998 to look at the new patch set (#4). Change subject: KUDU-2256: Add GetTransferSize() to RpcContext .. KUDU-2256: Add Ge

[kudu-CR] KUDU-2256: Add GetTransferSize() to RpcContext

2018-01-11 Thread Lars Volker (Code Review)
Hello Michael Ho, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8998 to look at the new patch set (#5). Change subject: KUDU-2256: Add GetTransferSize() to RpcContext .. KUD

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Hello Tidy Bot, David Ribeiro Alves, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8951 to look at the new patch set (#7). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 7: I've backed out the integration test, since it expo

[kudu-CR] data dirs: fix logging message

2018-01-11 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8999 ) Change subject: data_dirs: fix logging message .. data_dirs: fix logging message If fs_target_data_dirs_per_tablet is set to be g

[kudu-CR] [docs] Document how to recover from a majority failed tablet

2018-01-11 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/8402 ) Change subject: [docs] Document how to recover from a majority failed tablet .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8

[kudu-CR] [docs] Document how to recover from a majority failed tablet

2018-01-11 Thread Will Berkeley (Code Review)
Hello Mike Percy, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8402 to look at the new patch set (#6). Change subject: [docs] Document how to recover from a majority failed tablet

[kudu-CR](branch-1.5.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Hello David Ribeiro Alves, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8941 to look at the new patch set (#8). Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo ..

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Patch Set 8: (2 comments) > (3 comments)

[kudu-CR] [docs] Document how to recover from a majority failed tablet

2018-01-11 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/8402 ) Change subject: [docs] Document how to recover from a majority failed tablet .. Patch Set 6: I wonder if we could automate the whole th

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Hao Hao (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9004 to look at the new patch set (#2). Change subject: KUDU-2254: Detect and warn about misusage of KuduContext

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/90

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 7: Code-Review+2 -- To view, visit http://ger

[kudu-CR] [docs] Added a section for Kudu clients connecting to secure clusters

2018-01-11 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/8953 ) Change subject: [docs] Added a section for Kudu clients connecting to secure clusters .. Patch Set 1: (4 comments) > (4 comments) done

[kudu-CR] rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7821 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 2: (1 comment) http://gerrit.cloudera.or

[kudu-CR] rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7821 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:

[kudu-CR] rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7821 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 2: (1 comment) http://gerrit.cloudera.or

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8948 to look at the new patch set (#10). Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .

[kudu-CR] KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8951 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in RowSetInfo Th

[kudu-CR](branch-1.6.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9008 Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in RowSetInfo This overfl

[kudu-CR] rpc: allow setting --rpc tls min protocol on older RHEL versions

2018-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7821 ) Change subject: rpc: allow setting --rpc_tls_min_protocol on older RHEL versions .. Patch Set 2: (1 comment) http://gerrit.cloudera.or

[kudu-CR](branch-1.5.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8941 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 8: Two straight failures of what appears to be KUDU-17

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Patch Set 10: (1 comment) http:

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8948 to look at the new patch set (#11). Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Patch Set 10: (1 comment) > (1 comment) D

[kudu-CR](branch-1.5.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8941 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in RowSetI

[kudu-CR](branch-1.6.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9008 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. KUDU-2251: rowset size can overflow int in RowSetI

[kudu-CR](branch-1.6.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9008 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 1: Code-Review+2 -- To view, visit http://gerr

[kudu-CR](branch-1.5.x) KUDU-2251: rowset size can overflow int in RowSetInfo

2018-01-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/8941 ) Change subject: KUDU-2251: rowset size can overflow int in RowSetInfo .. Patch Set 8: Code-Review+2 -- To view, visit http://gerr

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8948 to look at the new patch set (#12). Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8948 to look at the new patch set (#13). Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-11 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Patch Set 13: > Uploaded patch set 13. Cha

[kudu-CR] Kudu-2208 Avoid system call interruption in Subprocess

2018-01-11 Thread Jeffrey F. Lukman (Code Review)
Jeffrey F. Lukman has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9011 Change subject: Kudu-2208 Avoid system call interruption in Subprocess .. Kudu-2208 Avoid system call interruption in Subprocess Thi

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/

[kudu-CR] Kudu-2208 Avoid system call interruption in Subprocess

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9011 ) Change subject: Kudu-2208 Avoid system call interruption in Subprocess .. Patch Set 1: (10 comments) I think it's also worth testing Sub

[kudu-CR] KUDU-2253 Deltafile on-disk size is 3x larger than expected

2018-01-11 Thread Dan Burkert (Code Review)
Hello Will Berkeley, Kudu Jenkins, Grant Henke, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8982 to look at the new patch set (#2). Change subject: KUDU-2253 Deltafile on-disk size is 3x larger than expected .

[kudu-CR] KUDU-2253 Deltafile on-disk size is 3x larger than expected

2018-01-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8982 ) Change subject: KUDU-2253 Deltafile on-disk size is 3x larger than expected .. Patch Set 2: New version should be backwards compatible wi

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/900

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: I would like to abandon this code review if there is

[kudu-CR] KUDU-2253 Deltafile on-disk size is 3x larger than expected

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8982 ) Change subject: KUDU-2253 Deltafile on-disk size is 3x larger than expected .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: Hmm, I thought we'd seen a case where a user had

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: > Hmm, I thought we'd seen a case where a user had c

[kudu-CR] consensus: Fix KUDU-1735 regression test in 3-4-3 mode

2018-01-11 Thread Mike Percy (Code Review)
Hello Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9013 to review the following change. Change subject: consensus: Fix KUDU-1735 regression test in 3-4-3 mode .. consensus:

[kudu-CR] KUDU-2208 Refine unit test and move RETRY ON EINTR() to os-util.h

2018-01-11 Thread Jeffrey F. Lukman (Code Review)
Jeffrey F. Lukman has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9015 Change subject: KUDU-2208 Refine unit test and move RETRY_ON_EINTR() to os-util.h .. KUDU-2208 Refine unit test and move RETRY_ON_EI

[kudu-CR] build: Move fake XML file generation to run-test.sh

2018-01-11 Thread Mike Percy (Code Review)
Hello Edward Fancher, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8984 to look at the new patch set (#3). Change subject: build: Move fake XML file generation to run-test.sh ..

[kudu-CR] KUDU-2216. Post process gtest generated xml to include the output from the *.txt files

2018-01-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8757 ) Change subject: KUDU-2216. Post process gtest generated xml to include the output from the *.txt files .. Patch Set 14: Consider using ht

[kudu-CR] WIP [raft consensus-itest] proper fix for Test KUDU 1735

2018-01-11 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8989 ) Change subject: WIP [raft_consensus-itest] proper fix for Test_KUDU_1735 .. Patch Set 2: I posted a rough sketch of a fix and an explanati

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

2018-01-11 Thread Hao Hao (Code Review)
Hello 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 (#2). Change subject: [WIP]KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

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

2018-01-11 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Mike Percy, David Ribeiro Alves, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8804 to look at the new patch set (#5). Change subject: [WIP] KUDU-1704: add READ_YOUR_WRITES scan mode .

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

2018-01-11 Thread Hao Hao (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8847 to look at the new patch set (#2). Change subject: [WIP]KUDU-1704: add java client support for READ_YOUR_WRITES mode ..

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8847 ) Change subject: [WIP]KUDU-1704: add java client support for READ_YOUR_WRITES mode .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8804 ) Change subject: [WIP] KUDU-1704: add READ_YOUR_WRITES scan mode .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/8804/1/src/k

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: [WIP]KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:808

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

2018-01-11 Thread Hao Hao (Code Review)
Hello Alexey Serbin, Mike Percy, David Ribeiro Alves, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8804 to look at the new patch set (#6). Change subject: [WIP] KUDU-1704: add READ_YOUR_WRITES scan mode .

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

2018-01-11 Thread Hao Hao (Code Review)
Hello 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 (#3). Change subject: [WIP] KUDU-1704: add c++ client support for READ_YOUR_WRITES mode ..

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

2018-01-11 Thread Hao Hao (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8847 to look at the new patch set (#3). Change subject: [WIP] KUDU-1704: add java client support for READ_YOUR_WRITES mode .

[kudu-CR] KUDU-2256: Add GetTransferSize() to RpcContext

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8998 ) Change subject: KUDU-2256: Add GetTransferSize() to RpcContext .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.or

[kudu-CR] KUDU-2256: Add GetTransferSize() to RpcContext

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8998 ) Change subject: KUDU-2256: Add GetTransferSize() to RpcContext .. KUDU-2256: Add GetTransferSize() to RpcContext This changes add

[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9004 ) Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. Patch Set 2: Got it. Sounds reasonable to abandon this then,

[kudu-CR] KUDU-2208 Refine unit test and move RETRY ON EINTR() to os-util.h

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9015 ) Change subject: KUDU-2208 Refine unit test and move RETRY_ON_EINTR() to os-util.h .. Patch Set 1: Can you merge this into the previous c

[kudu-CR] [tls socket-test] fix test to pass on Ubuntu 16.04

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8996 ) Change subject: [tls_socket-test] fix test to pass on Ubuntu 16.04 .. [tls_socket-test] fix test to pass on Ubuntu 16.04 Prior to

[kudu-CR] [tls socket-test] fix test to pass on Ubuntu 16.04

2018-01-11 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8996 ) Change subject: [tls_socket-test] fix test to pass on Ubuntu 16.04 .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.clouder

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [WIP] KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/8823 To

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: [WIP] KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 3: Verified+1 Override it as the failed test

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has removed a vote on this change. Change subject: [WIP] KUDU-1704: add java client support for READ_YOUR_WRITES mode .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/8847 To

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

2018-01-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8847 ) Change subject: [WIP] KUDU-1704: add java client support for READ_YOUR_WRITES mode .. Patch Set 3: Verified+1 Override it as the failed test

[kudu-CR] Kudu-2208 Avoid system call interruption in Subprocess

2018-01-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9011 ) Change subject: Kudu-2208 Avoid system call interruption in Subprocess .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/

[kudu-CR] KUDU-2208 Add unit test to detect Subprocess Interruption Handling failure and patch to fix the bug

2018-01-11 Thread Jeffrey F. Lukman (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9015 to look at the new patch set (#2). Change subject: KUDU-2208 Add unit test to detect Subprocess Interruption Handling failure and patch to fix the bug