Andrew Sherman has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11459 )

Change subject: IMPALA-589: Add sql function returning the impalad coordinator 
hostname.
......................................................................


Patch Set 2:

(1 comment)

Thanks for making me look at the *.test files

http://gerrit.cloudera.org:8080/#/c/11459/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11459/2//COMMIT_MSG@7
PS2, Line 7: IMPALA-589: Add sql function returning the impalad coordinator 
hostname.
> I appreciate your thoroughness, but we generally make our commit messages a
Done



--
To view, visit http://gerrit.cloudera.org:8080/11459
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I94d6e2664ba659b48df53c5c06f67b502c533e47
Gerrit-Change-Number: 11459
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman <asher...@cloudera.com>
Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Thomas Marshall <thomasmarsh...@cmu.edu>
Gerrit-Reviewer: Zoram Thanga <zo...@cloudera.com>
Gerrit-Comment-Date: Mon, 24 Sep 2018 16:09:12 +0000
Gerrit-HasComments: Yes

Reply via email to