[kudu-CR] [java-client] add missing check for open client

2017-01-03 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] add missing check for open client .. Patch Set 1: > Somehow the buffers are not flushed by the time the close() method > returns, and from a

[kudu-CR] [java-client] add missing check for open client

2016-11-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java-client] add missing check for open client .. Patch Set 1: Ah... I do agree that close() combined with an 'autoflush' mode should assume that all written dat

[kudu-CR] [java-client] add missing check for open client

2016-11-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] add missing check for open client .. Patch Set 1: This is an interesting failure. The test (https://github.com/apache/kudu/blob/master/java/kudu-cl

[kudu-CR] [java-client] add missing check for open client

2016-11-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [java-client] add missing check for open client .. Patch Set 1: The patch looks reasonable, but seems to have caused some test failure. Is there a cross-method de

[kudu-CR] [java-client] add missing check for open client

2016-11-22 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5190 to review the following change. Change subject: [java-client] add missing check for open client .. [java-client