advancedxy commented on PR #11842: URL: https://github.com/apache/iceberg/pull/11842#issuecomment-2560757725
> > For any relative new clients(supports v2 format), it should produces specs with field id included. > > It should indeed, but you cannot guarantee that, and it is not enforced by the spec. Hmmm, maybe I'm not following here. When it's upgraded to v2, I think the table metadata should be regenerated with partition field id included? And the spec enforces that the field id should be unique across all the specs: https://iceberg.apache.org/spec/#partitioning <img width="826" alt="image" src="https://github.com/user-attachments/assets/5ef520da-8059-4d9a-8b38-ceda5f3aa6b6"; /> -- 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]
