EmilyMatt commented on PR #15613: URL: https://github.com/apache/datafusion/pull/15613#issuecomment-2786734652
> Are we planning to deprecate MemoryConsumer and replace it all with TrackedConsumer ? I think TrackedConsumer is only relevant for the TrackConsumerPool, it doesn't exist anywhere else, most pools don't track consumers at all, it is up to users to implement a MemoryPool that takes into consideration each Consumer and how much it has requested. This API is only meant to help users do that, as the current API does not really allow doing that with certainty(there can be surrounding hacks to achieve that functionality, but they far from perfect without some unique identifier) -- 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-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
