Skip to content

Commit

Permalink
[java client] Small fix to logging in AsyncKuduClient.tooManyAttempts…
Browse files Browse the repository at this point in the history
…OrTimeout

We were logging the RPC after calling errback on it, so some of the fields were
already nulled out. Mostly a cosmetic change, for a DEBUG log line.

Change-Id: If3df93be813c7fdf1626a06df14ff9b7585655fd
Reviewed-on: http://gerrit.cloudera.org:8080/4948
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
  • Loading branch information
jdcryans authored and adembo committed Nov 4, 2016
1 parent a833a15 commit 4396659
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -949,8 +949,8 @@ static <R> Deferred<R> tooManyAttemptsOrTimeout(final KuduRpc<R> request,
}
Status statusTimedOut = Status.TimedOut(message + request);
final Exception e = new NonRecoverableException(statusTimedOut, cause);
request.errback(e);
LOG.debug("Cannot continue with this RPC: {} because of: {}", request, message, e);
request.errback(e);
return Deferred.fromError(e);
}

Expand Down

0 comments on commit 4396659

Please sign in to comment.