gaborkaszab commented on PR #17545: URL: https://github.com/apache/iceberg/pull/17545#issuecomment-5701771328
Hey @RussellSpitzer , You were right, I managed to get rid of the new interface by introducing versions of functions that accept location and keyId instead of these classes/interfaces. Thanks for the fresh pair of eyes! @stevenzwu I think this PR is still something you can build the Snapshot.rootLocation abstraction on. -- 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]
