Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13308 )
Change subject: [java] Fix handling of SERVICE_UNAVAILABLE errors ...................................................................... [java] Fix handling of SERVICE_UNAVAILABLE errors When the Java client sends a write, and it is rejected due to, e.g., a soft memory limit error, it causes the client to invalidate the leader's location for the tablet. The client then has to do a GetTableLocations lookup on the master to refresh the cache. This is silly because the location is valid and the leader is where the write must go. This patch corrects the handling of certain RPC-level errors including the handling of SERVICE_UNAVAILABLE. It will reduce greatly the number of unnecessary calls to the master. These usually aren't a problem because master lookups are fast, but it's a waste of time and resources nonetheless. Change-Id: Id3437c779322e756a6e1fbc19f464f765741d58b Reviewed-on: http://gerrit.cloudera.org:8080/13308 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Andrew Wong <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java 2 files changed, 14 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, but someone else must approve Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13308 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id3437c779322e756a6e1fbc19f464f765741d58b Gerrit-Change-Number: 13308 Gerrit-PatchSet: 2 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]>
