szehon-ho edited a comment on pull request #2240: URL: https://github.com/apache/iceberg/pull/2240#issuecomment-787059052
Thanks guys for the time and discussion (learning a lot on the side). If I understand correctly, it looks metrics-promotion on read is the way to go due to simplicity, esp in regards to alter table not having to alter auto-promoted metrics properties. But in this approach we have to manually pass in sortOrder in all places of write, and I missed it in the new SparkTableUtil and Flink, which I can add. I'll also take a look at how hard it is to serialize the SortOrder to the writers. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
