ajantha-bhat commented on PR #10176: URL: https://github.com/apache/iceberg/pull/10176#issuecomment-2316864310
> Add PartitionStatsWriter that would wrap PartitionStats as Record This abstraction isn't working out. Because `PartitionStats` need partition as `PartitionData` and `Record` need partition as `Record`. Converting them on wrapping and unwrapping is becoming very messy because storage is same and there are multiple paths to store it. I am thinking to introduce `PartitionStatsRecord` that extends `Record` and store it in `Core` similar to `GenericRecord` and make the interfaces based on that in `core` and `data` instead of `PartitionStats`. Thoughts? -- 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
