rdblue commented on PR #6701: URL: https://github.com/apache/iceberg/pull/6701#issuecomment-1410958514
The reason why this wasn't in the original spec is because we can technically update it by getting the highest assigned ID from the new schema. Because it didn't strictly need to be part of the spec here, we omitted it. Still, we send this in the actual implementation because it simplifies the server side. I'm ambivalent about whether we actually should include this in the spec. Keeping the spec simple is usually the best option. But I can see the argument for doing this. Maybe we should update the test service to no longer require the `last-column-id`. FYI @bryanck. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
