Alexey Serbin has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/6401

Change subject: [java client] fixed NPE in master RPC error handler
......................................................................

[java client] fixed NPE in master RPC error handler

Fixed an NPE in dispatchMasterErrorOrReturnException().  The
AsyncKuduClient.handleNotLeader() is used to handle both tablet-
and master server errors.  In the latter case, the KuduRpc.tablet member
is null, so accessing it results in NPE with exception stack like below:

java.lang.NullPointerException
  at 
org.apache.kudu.client.AsyncKuduClient.handleNotLeader(AsyncKuduClient.java:1269)
  at 
org.apache.kudu.client.TabletClient.dispatchMasterErrorOrReturnException(TabletClient.java:536)
  at org.apache.kudu.client.TabletClient.messageReceived(TabletClient.java:461)
  at 
org.jboss.netty.channel.SimpleChannelUpstreamHandler.handleUpstream(SimpleChannelUpstreamHandler.java:70)
  at org.apache.kudu.client.TabletClient.handleUpstream(TabletClient.java:595)
  at 
org.jboss.netty.channel.DefaultChannelPipeline.sendUpstream(DefaultChannelPipeline.java:564)
  ...

The error has been spotted while running the
org.apache.kudu.client.TestRowErrors test.

Change-Id: Ib923212db1e0931ed63b31a39f03335c701f91cc
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
1 file changed, 6 insertions(+), 1 deletion(-)


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

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

Reply via email to