gianm commented on issue #6588: autosize processing buffers based on direct memory sizing by default URL: https://github.com/apache/incubator-druid/pull/6588#issuecomment-438456745 I haven't seen 2GB buffers be super useful except in niche cases (really big aggregator states). So it might be better to have the cap be 1GB, which is the current default, which means nobody should run into the problem @nishantmonu51 brought up.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org