Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23218 )
Change subject: [tserver] more detailed trace on early scan response ...................................................................... [tserver] more detailed trace on early scan response This patch adds more details into the traces emitted on early scan responses. I also sneaked in a fix for a potential integer overflow in ColumnarResultSerializer::ResponseSize() when using larger column cells. Also, a non-OK status from Iterator::NextBlock() is now logged as error, not warning, since it's expected to error out only in case of corruption and other unexpected server-side conditions. This changelist doesn't contain any functional modifications. Change-Id: Ifb171fcf46b4d8534e7d8f933b3b0594f940ace1 Reviewed-on: http://gerrit.cloudera.org:8080/23214 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit 73a4adc889863e3194fe97d7f75cc1198dfa4569) Reviewed-on: http://gerrit.cloudera.org:8080/23218 Reviewed-by: Marton Greber <[email protected]> Tested-by: Marton Greber <[email protected]> --- M src/kudu/tserver/tablet_service.cc 1 file changed, 15 insertions(+), 14 deletions(-) Approvals: Marton Greber: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23218 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: Ifb171fcf46b4d8534e7d8f933b3b0594f940ace1 Gerrit-Change-Number: 23218 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]>
