Tim Armstrong has posted comments on this change.

Change subject: Reduce the impalad mem limit for the mini-cluster
......................................................................


Patch Set 1:

(1 comment)

Maybe run an exhaustive private build?

It might be a little tricky to get this right: I've seen situations where one 
impalad (perhaps the coordinator) has significantly higher memory usage than 
the others, but I think it will be good.

http://gerrit.cloudera.org:8080/#/c/2472/1/bin/start-impala-cluster.py
File bin/start-impala-cluster.py:

Line 219: virtual_memory().total
It's confusing that virtual_memory().total returns the total physical memory. 
Maybe clarify in the comment that it's the total physical memory (when I was 
initially reading the code I was very confused).


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I468355133fcceec12a8c7e8b4f46bf9c932aaf2c
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to