Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/19388 )
Change subject: IMPALA-11375 Impala shell outputs details of each RPC ...................................................................... Patch Set 11: (2 comments) addressed comments http://gerrit.cloudera.org:8080/#/c/19388/11//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19388/11//COMMIT_MSG@2 PS11, Line 2: Author: jasonmfehr <jasonmf...@gmail.com> > Nit: Author still shows gmail address I was going to fix this on future gerrit changes only, but since there is another code change needed on this PR I am going to amend the commit and fix the email. http://gerrit.cloudera.org:8080/#/c/19388/11/shell/option_parser.py File shell/option_parser.py: http://gerrit.cloudera.org:8080/#/c/19388/11/shell/option_parser.py@216 PS11, Line 216: "file. Ignored if protocol is beeswax.") > Nit: delete double space Done -- To view, visit http://gerrit.cloudera.org:8080/19388 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I36f8dbc96726aa2a573133acbe8a558299381f8b Gerrit-Change-Number: 19388 Gerrit-PatchSet: 11 Gerrit-Owner: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Comment-Date: Thu, 05 Jan 2023 21:15:50 +0000 Gerrit-HasComments: Yes