Adar Dembo has submitted this change and it was merged. Change subject: client: don't log retriable scanner errors ......................................................................
client: don't log retriable scanner errors Retriable scanner errors are a normal part of life. Prior to this patch, every such error (eg SERVER_TOO_BUSY) resulted in both an INFO and a WARNING message logged in the client. In an Impala stress scenario this was producing several hundred logs per second, even while queries were mostly succeeding. This downgrades them to a VLOG(1) level. Change-Id: I32e16dd10f9de8a015d420da4b58b2f363045f67 Reviewed-on: http://gerrit.cloudera.org:8080/5380 Tested-by: Kudu Jenkins Reviewed-by: Jean-Daniel Cryans <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/client/client.cc M src/kudu/client/scanner-internal.cc 2 files changed, 9 insertions(+), 7 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5380 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32e16dd10f9de8a015d420da4b58b2f363045f67 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins
