[I] MemoryReservation should expose MemoryConsumer [arrow-datafusion]

["milenkovicm (via GitHub)" <gi...@apache.org>]

milenkovicm opened a new issue, #7999:
URL: https://github.com/apache/arrow-datafusion/issues/7999

   ### Is your feature request related to a problem or challenge?
   
   At the moment it looks impossible to implement new `MemoryManager` outside of datafusion crate as it is impossible to retrieve `MemoryConsumer`. `MemoryConsumer` is needed to check if consumer `can_spill` 
   
   ### Describe the solution you'd like
   
   A new method `pub fn consumer(&self) -> &MemoryConsumer` to be implemented in `MemoryReservation` exposing reference to the `MemoryConsumer`
   
   ### Describe alternatives you've considered
   
   I have considered exposing `Arc<SharedRegistration>` instead of `MemoryConsumer` but it does not have any usefull additional information 
   
   ### Additional context
   
   Relates to #7858


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] MemoryReservation should expose MemoryConsumer [arrow-datafusion]

["alamb (via GitHub)" <gi...@apache.org>]

alamb closed issue #7999: MemoryReservation should expose MemoryConsumer
URL: https://github.com/apache/arrow-datafusion/issues/7999


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org