rdblue commented on pull request #1505: URL: https://github.com/apache/iceberg/pull/1505#issuecomment-698491605
Would option 3 work? The reflection-based loader would need to be in the Hive classpath at all times. So we would have the same problem with that one right? This is a problem that Hive would need to fix. I lean toward the approach of making it optional to set the serde and storage handler, and using either reflection to see if the classes currently exist or a table property to change the behavior. A table property makes sense to me: `engine.hive.enabled=true` could signal that these should be real classes. What do you think? ---------------------------------------------------------------- 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]
