ajantha-bhat commented on PR #10176: URL: https://github.com/apache/iceberg/pull/10176#issuecomment-2311678409
> Add a wrapper class to represent partition stats in core. I didn't use the wrapper class initially and directly used record because for each entry having two objects (wrapper and Record) might increase memory footprint. But looking at benchmarks, it may not be a deal breaker and it will look clean as you suggested. I will update the code as you suggested. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
