Alexey Serbin has posted comments on this change.

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


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/6401/1//COMMIT_MSG
Commit Message:

Line 24: org.apache.kudu.client.TestRowErrors test.
> any ideas how we might provoke it to happen more reliably? Are we lacking t
Sure -- this issue became apparent when testing against changes for KUDU-1927 
(https://gerrit.cloudera.org/#/c/6356/)

I think we can provoke it if having test like I wrote for KUDU-1927.  Perhaps, 
we should just run an external minicluster with those parameters while running 
Java tests.  I don't know what is the best way of doing that -- just some 
additional maven profile for the project?


-- 
To view, visit http://gerrit.cloudera.org:8080/6401
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib923212db1e0931ed63b31a39f03335c701f91cc
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to