rdblue commented on pull request #1508:
URL: https://github.com/apache/iceberg/pull/1508#issuecomment-701059248


   I posted in a comment above, but I'll echo it here: I don't think we should 
be adding anything to the `Table` API that we are going to want to remove 
later. Since we don't have schema ids, I don't think we can add the API that we 
eventually want right now. Adding `Snapshot.schema` seems like something we can 
do earlier and that would be the most friendly API long term. That's why I 
suggest it. We can do something else, as long as we are careful and only add 
non-public methods.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to