Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] scanNextRows() needs to always increment attempts ......................................................................
[java client] scanNextRows() needs to always increment attempts This bug was introduced in c4180bb, we call something that checks getSleepTimeForRpc but we don't increment the attempt. If the client is null or dead, on the first call, then the attempt will still be set to 0. Discovered in a TestAsyncKuduClient#testDisconnect failure. Change-Id: I8a726adb82ade80f241128e7c61091e6af2d9c13 Reviewed-on: http://gerrit.cloudera.org:8080/2973 Reviewed-by: Dan Burkert <[email protected]> Tested-by: Jean-Daniel Cryans --- M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Dan Burkert: Looks good to me, approved Jean-Daniel Cryans: Verified -- To view, visit http://gerrit.cloudera.org:8080/2973 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a726adb82ade80f241128e7c61091e6af2d9c13 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans
