alamb commented on issue #25804: URL: https://github.com/apache/datafusion/issues/25804#issuecomment-6705589153
> and it leaves one gap that I'd track here rather than grow that PR. I don't think this gap should hold 56; does that work for you? That makes sense to me It seems like this ticket is of the form "the memory accounting got better, but it still needs to improve" so unless there is some sort of clear regression I think it sounds like we should just continue to fix it. In general, I do think it is clear that we need some sort of better memory tracking scheme than all this ad hoc changes. Maybe we can wrap the builders / calculations into some structure to encapsulate it or something 🤔 -- 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]
