Henry Robinson has posted comments on this change.

Change subject: IMPALA-5364: Correct title of query locations table
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/6988/1//COMMIT_MSG
Commit Message:

Line 12: We should have per-backend reporting on the number of fragment 
instances
> It would be nice to have a JIRA for this and mention it here.
Sure, but it's a bigger scoping effort and I don't want to file a JIRA right 
now for something vague.


http://gerrit.cloudera.org:8080/#/c/6988/1/be/src/service/impala-server.cc
File be/src/service/impala-server.cc:

PS1, Line 858: host
> nit: Mention this change in the commit message too.
this is really a tiny change and doesn't affect anything. I don't think it's 
worth calling out in the commit msg - let me know if you feel strongly.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2fddd41500ada10eb1c12d92afcb96a82009857b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Henry Robinson <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-HasComments: Yes

Reply via email to