RussellSpitzer commented on pull request #2848: URL: https://github.com/apache/iceberg/pull/2848#issuecomment-899699795
I was just thinking if you are already making your own SparkTable and Catalog, why not just build inside the Iceberg Packages in your own code? Then you won't need to change the classes to public. My worry about this feature in general would be if we were serious, we would want to have it be at the core level. If we only changed the catalog + table implementation another framework (or non-custom iceberg version) could write to the table with whatever mode it liked. -- 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]
