[GitHub] [iceberg] rdblue commented on pull request #5177: Spec: Sort-order order-id is mandatory

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

rdblue commented on PR #5177:
URL: https://github.com/apache/iceberg/pull/5177#issuecomment-1172469461

   We do require this in the parser and it is probably best if we require it all the time.
   
   There are a couple of places where we don't technically need it because the order will be reassigned. For example, when interacting with a REST-based catalog, the catalog service will add the sort order to the current metadata and reassign the ID to ensure it doesn't conflict with concurrent operations. But it's always okay to have an order ID so I think we should go ahead and require it.


-- 
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-unsubscribe@iceberg.apache.org

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