You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/04/02 04:13:09 UTC

[kudu-CR] WIP: scanner should not retry a SCANNER_EXPIRED error without reopening

Hello Adar Dembo,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: WIP: scanner should not retry a SCANNER_EXPIRED error without reopening
......................................................................

WIP: scanner should not retry a SCANNER_EXPIRED error without reopening

This fixes yet another tight retry loop seen while testing Impala. A scanner
expired on the server side, and the client got the expiration error. However,
the current code sees that as retryable, but should know that it's the kind
of retry that must re-open the scanner.

WIP because we need a test for this.

Change-Id: I5e09849d5b749db958c3a3df89fd8de9947ab991
---
M src/kudu/client/client.cc
M src/kudu/client/scanner-internal.cc
M src/kudu/client/scanner-internal.h
3 files changed, 16 insertions(+), 6 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e09849d5b749db958c3a3df89fd8de9947ab991
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>