miR172 commented on pull request #3177: URL: https://github.com/apache/iceberg/pull/3177#issuecomment-940395263
Agree that S3 should not be known by Catalog. I'll move it out in our code. I have some change locally to the existing Catalog, which make it compatible with both plain-JDBC and Jooq impl. We can keep Jooq in our own code and contribute the common interfaces. @nssalian let's chat on slack and coordinate the work. -- 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]
