ajantha-bhat commented on PR #6461:
URL: https://github.com/apache/iceberg/pull/6461#issuecomment-1382677828

   > I'm still a little worried about saving this information, what does 
knowing the sort order mean for a file. Are we guaranteeing that the file is 
locally sorted by that order? or globally sorted with files added in that 
snapshot. I'm cautious about doing anything here until we really define what it 
means.
   
   **Spec says this. I am just trying to make the implementation the same as 
the spec. I am ok with changing the spec or implementation.** 
   
   > `140 sort_order_id` - ID representing sort order for this file [3].
   If the sort order ID is missing or unknown, then the order is assumed to be 
unsorted. Only data files and equality delete files should be written with a 
non-null order id. [Position 
deletes](https://iceberg.apache.org/spec/#position-delete-files) are required 
to be sorted by file and position, not a table order, and should set sort order 
id to null. Readers must ignore sort order id for position delete files.
   
   cc: @rdblue 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to