You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org> on 2016/05/05 22:53:46 UTC

[kudu-CR] [java client] scanNextRows() needs to always increment attempts

Jean-Daniel Cryans has uploaded a new change for review.

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

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
---
M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java
1 file changed, 3 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a726adb82ade80f241128e7c61091e6af2d9c13
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans