Matthew Jacobs has posted comments on this change.

Change subject: IMPALA-5641: mem-estimate should never be less than 
mem-reservation
......................................................................


Patch Set 1:

Was there a reason not to fold this change into the patch for max_row_size? I 
guess to not change as many test files? Either way seems fine. I'll review the 
other patch shortly.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1e2853300371e31b13d81a763dbafb21709b16c4
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to