minleejae commented on PR #16927:
URL: https://github.com/apache/iceberg/pull/16927#issuecomment-4774954135
Hi @Fokko, could you take a look when you have a chance?
I think this is a follow-up to #11514 and #13850. The original direction
was to stop exposing `AddSchemaUpdate.last-column-id`, but the Java serializer
still sends the
deprecated optional field using `schema.highestFieldId()`. That can be
incorrect after dropping the column with the highest field ID.
I added a REST server apply-path test to verify that omitting the field
preserves the table-level `last-column-id`.
--
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]
