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

Reply via email to