kbendick commented on issue #3044: URL: https://github.com/apache/iceberg/issues/3044#issuecomment-907524109
If the only reason to have the GlueCatalog implement `Configurable` is to satisfy the current constraints of dynamic FileIO loading, likely we can change that (I believe dynamic FileIO loading was added when maybe the second additional Catalog beyond the basic Hadoop and Hive ones were added). In fact, on first look, it seems like the GlueCatalog could _possibly_ just pass in `null` for the `hadoopConf` if it didn't implement Configurable, as the current dynamic FileIO loading only sets the hadoop configuration if the class implements `Configurable`. https://github.com/apache/iceberg/blob/4eb0853cd787bf2f5778195558d22a45ecf6c601/core/src/main/java/org/apache/iceberg/CatalogUtil.java#L189-L191 @jackye1995 do you know if GlueCatalog really requires anything from the hadoop configuration? Or have any input on how Flink can be run in Kinesis Data Analytics? -- 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]
