Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13740 )
Change subject: [WIP] IMPALA-7486: Prototype for admitting less mem on coord ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/13740/7//COMMIT_MSG Commit Message: PS7: I had a couple of higher-level thoughts: * It looks like the estimates are in some cases higher on the coordinator than executors, I think because of the padding. I think this is probably mostly OK - there's definitely more overhead on the coordinator if the fragments are tiny. * We probably want a "chicken bit" to fall back to the old behaviour in case there are regressions. -- To view, visit http://gerrit.cloudera.org:8080/13740 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2b94e7293b91dec8a18491079c34923eadd94b21 Gerrit-Change-Number: 13740 Gerrit-PatchSet: 7 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Mon, 22 Jul 2019 07:39:09 +0000 Gerrit-HasComments: Yes
