Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/19932 )
Change subject: IMPALA-12163: Fixes an issue when outputting RPC details where the start and end times could be longer than the call duration. Additionally, the end time of the exact same rpc call was different between stdout and the rpc details file. ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/19932/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19932/1//COMMIT_MSG@7 PS1, Line 7: Fixes an issue when outputting RPC details where the nit: the title should be a separate line http://gerrit.cloudera.org:8080/#/c/19932/1/tests/shell/test_shell_commandline.py File tests/shell/test_shell_commandline.py: http://gerrit.cloudera.org:8080/#/c/19932/1/tests/shell/test_shell_commandline.py@1497 PS1, Line 1497: "rpc detail output not supported with beeswax protocol") nit: +2 indentation http://gerrit.cloudera.org:8080/#/c/19932/1/tests/shell/test_shell_commandline.py@1523 PS1, Line 1523: "difference found between stdout and rpc file:\nSTDOUT:\n{0}\nFILE:\n{1}" \ : .for nit: +2 indentation -- To view, visit http://gerrit.cloudera.org:8080/19932 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifd9dec189d0f6fb8713fb1c7b2b6c663e492ef05 Gerrit-Change-Number: 19932 Gerrit-PatchSet: 1 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Thu, 25 May 2023 08:56:22 +0000 Gerrit-HasComments: Yes
