andygrove opened a new issue, #1449: URL: https://github.com/apache/datafusion-comet/issues/1449
### Describe the bug As demonstrated in unit tests added in https://github.com/apache/datafusion-comet/pull/1440, we are allocating ~100kb for a batch when the actual memory used in less than 10kb. ``` assert_eq!(8376, batch.get_array_memory_size()); assert_eq!(106496, buffer.reservation.size()); ``` Rather than use the current `slot_size` logic that produces an estimate based on schema, it would be more accurate to estimate memory size based on the memory used by the incoming batches. ### Steps to reproduce _No response_ ### Expected behavior _No response_ ### Additional context _No response_ -- 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.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
