[GitHub] [iceberg] aokolnychyi edited a comment on pull request #2214: Add sort order to writer classes

[GitBox <gi...@apache.org>]

aokolnychyi edited a comment on pull request #2214:
URL: https://github.com/apache/iceberg/pull/2214#issuecomment-796342152


   cc @szehon-ho  @RussellSpitzer @rdblue who participated in #2240.
   
   This PR has been open for a while. I think it has a great basis for propagating the sort order to writers but I am afraid we cannot mark written files with the current sort order unless we sure those files are sorted. That will require a query engine API to request a distribution and ordering. For Spark, it will be possible only in 3.2. What about the other query engine integrations, @openinx @pvary?
   
   If my understanding is correct, what about limiting the scope of this PR to just propagate the sort order to 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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org