Improve memory pool reservation `shrink` error handling
Open
- Dominant language
- Rust
- Stars
- 9.3k
- Forks
- 2.4k
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 344
Description
I wonder in general if we should remove shrink and instead always use try_shrink (in a different PR of course) to catch any accounting bugs 🤔
_Originally posted by @alamb in https://github.com/apache/datafusion/pull/11218#discussion_r1676552228_
Contributor guide
Assessment
This issue has not been assessed yet.