danielcweeks commented on PR #473: URL: https://github.com/apache/iceberg-python/pull/473#issuecomment-1969374458
The [spec calls out](https://iceberg.apache.org/spec/#table-metadata-fields) the `current_snapshot_id` as optional, so omitting it "should" be valid. I think I agree with @Fokko that we shouldn't just blindly follow java's implementation. I would suggest a compatibility flag that sets the value to -1, but default it to off. The reason I think this is safe is that the situation where someone is using an older version of java but also using python to create tables (no commits) is relatively narrow and they would identify the problem quickly and turn the flag on for their use cases. I feel like this is the best approach to move forward with the correct solution, but still accommodate backward compatibility. -- 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
