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] add missing check for open client

The check was missing in the somewhat-public table location APIs.
Without the check the attached test times out because the associated RPC
is never completed.  I'm not sure if that's a bug, or by design.

Change-Id: I405d85a1295bf4e4aac348942d12efba40953a91
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
2 files changed, 12 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/90/5190/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5190
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I405d85a1295bf4e4aac348942d12efba40953a91
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>

Reply via email to