wiedld commented on code in PR #11665:
URL: https://github.com/apache/datafusion/pull/11665#discussion_r1692419610
##########
datafusion/execution/src/memory_pool/pool.rs:
##########
@@ -231,6 +235,10 @@ impl MemoryPool for FairSpillPool {
}
}
+/// Constructs a resources error based upon the individual
[`MemoryReservation`].
+///
+/// The error references the total bytes affiliated with the reservation's
+/// [`MemoryConsumer`], and not the total within the collective [`MemoryPool`].
#[inline(always)]
fn insufficient_capacity_err(
Review Comment:
In a follow up PR, I would like to iterate on this error message.
Take the original/current:
`Failed to allocate additional {} bytes for {} with {} bytes already
allocated - maximum available is {}`
And change into something like:
`Failed to allocate additional {} bytes for {} with {} bytes already
allocated for this reservation - maximum available is {} for the total pool`
--
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
