Skip to content

Commit

Permalink
fix(11523): remove to_root() for the error, since the immediate inner…
Browse files Browse the repository at this point in the history
… child should be returning an OOM
  • Loading branch information
wiedld committed Jul 26, 2024
1 parent a8383fa commit 1b1223f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions datafusion/execution/src/memory_pool/pool.rs
Original file line number Diff line number Diff line change
Expand Up @@ -362,16 +362,16 @@ impl<I: MemoryPool> MemoryPool for TrackConsumersPool<I> {
}

fn try_grow(&self, reservation: &MemoryReservation, additional: usize) -> Result<()> {
self.inner.try_grow(reservation, additional).map_err(|e| {
match e.find_root() {
self.inner
.try_grow(reservation, additional)
.map_err(|e| match e {
DataFusionError::ResourcesExhausted(e) => {
DataFusionError::ResourcesExhausted(
e.to_owned() + ". " + &self.report_top(),
)
}
_ => e,
}
})?;
})?;

self.tracked_consumers
.lock()
Expand Down

0 comments on commit 1b1223f

Please sign in to comment.