erenavsarogullari opened a new issue, #20386:
URL: https://github.com/apache/datafusion/issues/20386

   ### Is your feature request related to a problem or challenge?
   
   `memory_limit` (`RuntimeEnvBuilder::new().with_memory_limit()`) 
configuration uses `greedy` memory pool as `default`. However, if `memory_pool` 
(`RuntimeEnvBuilder::new().with_memory_pool()`) is set, it overrides by 
expected `memory_pool` config such as `fair`. Also, if both `memory_limit` and 
`memory_pool` configs are not set, then `unbounded` memory pool is being used 
so it can be useful to expose `ultimately used/selected pool` as part of 
Resources Exhausted error message for the end user awareness and the user may 
need to switch used memory pool (`greedy, fair, unbounded`),
   - Also, [this comparison 
table](https://github.com/lance-format/lance/issues/3601#issuecomment-2752838168)
 is an example use-case  for both greedy and fair memory pools runtime 
behaviors and this addition can help for this kind of comparison table by 
exposing used memory pool info as part of native logs.
   
   ### Describe the solution you'd like
   
   - Adding name property to MemoryPool instances,
   - Expose used MemoryPool info to Resources Exhausted error messages.
   
   ### Describe alternatives you've considered
   
   _No response_
   
   ### Additional context
   
   _No response_


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