Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/22958


Change subject: WIP [kudu-client] fix NPE in TableLocationsCache.get()
......................................................................

WIP [kudu-client] fix NPE in TableLocationsCache.get()

It might happen that the special table location entry for Kudu master
is absent in the cache, but the code since [1] doesn't check for null
before calling isStale() on the item returned from the cache.

This patch addresses the issue.

WIP:
  * file corresponding JIRA item
  * add a reproduction test scenario to trigger NPE prior to this fix

This is a follow-up to [1].

[1] https://github.com/apache/kudu/commit/a50aa5bfd

Change-Id: Ic1bf9001eb88e85c2d9913b804d3f223ab022cdc
---
M java/kudu-client/src/main/java/org/apache/kudu/client/TableLocationsCache.java
1 file changed, 3 insertions(+), 3 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic1bf9001eb88e85c2d9913b804d3f223ab022cdc
Gerrit-Change-Number: 22958
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to