electrum opened a new pull request, #7049: URL: https://github.com/apache/iceberg/pull/7049
Using `Configurable<Configuration>` in catalog implementations requires that downstream users to have Hadoop on the classpath for compilation. This is a blocker for Trino to [decouple from Hadoop and Hive code bases](https://github.com/trinodb/trino/issues/15921). Without this change, `javac` fails when Hadoop is not on the classpath: ``` ERROR] /Users/dphillips/src/trino/plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/catalog/jdbc/TrinoJdbcCatalog.java:[121,24] cannot access org.apache.hadoop.conf.Configuration class file for org.apache.hadoop.conf.Configuration not found ``` Note that the line number seems fairly arbitrary. It is calling the `createNamespace()` method on `JdbcCatalog`. Removing that line simply makes it fail on a different call to `JdbcCatalog`. So the problem seems to be the reference to the Hadoop class in the generic type parameter of the interface list. -- 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]
