apache / datafusion

Apache DataFusion SQL Query Engine
https://datafusion.apache.org/
Apache License 2.0
5.83k stars 1.1k forks source link

Improve memory pool reservation `shrink` error handling #11457

Open comphead opened 1 month ago

comphead commented 1 month ago

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_

comphead commented 1 month ago

Depends on #11218