xinyiZzz commented on pull request #8548: URL: https://github.com/apache/incubator-doris/pull/8548#issuecomment-1081352888
> The purpose of the first 1/2 is to reserve half of exec_mem_limit for other components rather than block_mgr. The purpose of the second 1/2 is to trigger spill before block_mgr_limit is full. Maybe twice 1/2 seems to be conservative. The first 1/2 I understand, great~, but I'm not sure if 1/2 is a good value. The role of the second 1/2. Currently, `BufferedBlockMgr2` has a similar implementation. In `write_unpinned_blocks`, unpinned blocks will be spilled in advance to avoid waiting when full.  -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
