Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11645 )
Change subject: client: add tablet id to scanner error messages ...................................................................... client: add tablet id to scanner error messages Some of the existing scanner log messages are confusing because at some point the KuduScanner::Data::DebugString() method was changed to have different output. This patch cleans up the output so that the sentences make sense and so we always print the id of the tablet that returned the error, if it's available. Also fixed a couple of clang-tidy warnings. Change-Id: I349d624c87d924df81d34c5eeb556433eac7b440 Reviewed-on: http://gerrit.cloudera.org:8080/11645 Tested-by: Mike Percy <[email protected]> Reviewed-by: Will Berkeley <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/client/client.cc M src/kudu/client/scanner-internal.cc M src/kudu/client/scanner-internal.h 3 files changed, 20 insertions(+), 17 deletions(-) Approvals: Mike Percy: Verified Will Berkeley: Looks good to me, approved Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I349d624c87d924df81d34c5eeb556433eac7b440 Gerrit-Change-Number: 11645 Gerrit-PatchSet: 3 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Will Berkeley <[email protected]>
