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]
