MonkeyCanCode commented on PR #16521:
URL: https://github.com/apache/iceberg/pull/16521#issuecomment-4525289998

   > I'm curious what the expected behavior is with using this new feature, if 
a field is dropped or sort order spec is removed?
   
   Based on my understanding, the #15150 feature uses only the current default 
sort order at the write time. The historical orders only need to be loadable. 
This fix restore load-ability when a historical sort order's filed is dropped 
without the needed of re-enabling use of that dropped filed. 
   
   Then regarding sort order been removed, I don't think there is DDL to 
actually remove it as setting `WRITE UNORDER` just flipped the 
default-sort-order-id from 1 to 0. There is no `removeSortOrders` (there was a 
PR related to adding this support but not accepted: 
https://github.com/apache/iceberg/pull/13975) 


-- 
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