joroKr21 commented on code in PR #16359:
URL: https://github.com/apache/datafusion/pull/16359#discussion_r2168219555
##########
datafusion/execution/src/memory_pool/mod.rs:
##########
@@ -131,14 +133,58 @@ pub trait MemoryPool: Send + Sync + std::fmt::Debug {
/// This must always succeed
fn grow(&self, reservation: &MemoryReservation, additional: usize);
+ /// Infallibly grow the provided `reservation` by bytes in held in
&[Arc<dyn Array>]
+ ///
+ /// This defaults to summing the memory size of all arrays, but can be
+ /// overridden by implementations that track the memory size of Array
usages
+ fn grow_with_arrays(
+ &self,
+ reservation: &MemoryReservation,
+ arrays: &[Arc<dyn Array>],
Review Comment:
I don't think that works since `MemoryReservation` holds array refs, we
don't have record batch refs
--
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]
