xiaokang opened a new pull request #8548:
URL: https://github.com/apache/incubator-doris/pull/8548


   
   # Proposed changes
   
   Issue Number: NA
   
   ## Problem Summary:
   
   Sort spill does not work since spill is triggered when memory limit exceed 
but some more memory need to be allocated during spilling.
   
   This pr fix sort spill memory limit by:
   1.change block_mgr_limit from exec_mem_limit to half of block_mgr_limit
   2.trigger spill when 1/2 of block_mgr_limit is used, implemented in 
should_spill() function
   ## Checklist(Required)
   
   1. Does it affect the original behavior: (No)
   2. Has unit tests been added: (No)
   3. Has document been added or modified: (No Need)
   4. Does it need to update dependencies: (No)
   5. Are there any changes that cannot be rolled back: (No)
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[[email protected]](mailto:[email protected]) by explaining why you 
chose the solution you did and what alternatives you considered, etc...
   


-- 
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]

Reply via email to