Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10584 )
Change subject: IMPALA-7077. Add a configuration for the max number of partitions to load ...................................................................... Patch Set 1: Another idea that also came up was to go with a flag like the one in this change, but base the limit on memory consumed. That would cover more scenarios for banning a table: num partitions, num files, num columns, incremental stats, or some combination. We can obtain the size estimate at the per partition granularity, so once exceeded, back out as in this change. There's still the downside of a static flag setting (less flexibility) but there is some ease-of-use that can be argued as well. -- To view, visit http://gerrit.cloudera.org:8080/10584 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ife99a97a891ed14675303ea472abb2932a72cb51 Gerrit-Change-Number: 10584 Gerrit-PatchSet: 1 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Tue, 05 Jun 2018 22:07:54 +0000 Gerrit-HasComments: No
