You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "nandorKollar (via GitHub)" <gi...@apache.org> on 2026/02/17 12:13:58 UTC

[PR] fix: HMS federation doesn't build due to outdated depdency [polaris]

nandorKollar opened a new pull request, #3795:
URL: https://github.com/apache/polaris/pull/3795

   <!--
   ๐Ÿ“ Describe what changes you're proposing, especially breaking or user-facing changes. 
   ๐Ÿ“– See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
   -->
   
   Fix the broken HMS federation build (`-PNonRESTCatalogs=HIVE`). It seems that there's an outdated, very old asm dependency which isn't compatible with Java 21, that's why the build is failing. This PR excludes the problematic dependency, which is brought in by Hive 3.1.3.
   
   ## Checklist
   - [ ] ๐Ÿ›ก๏ธ Don't disclose security issues! (contact security@apache.org)
   - [ ] ๐Ÿ”— Clearly explained why the changes are needed, or linked related issues: Fixes #
   - [ ] ๐Ÿงช Added/updated tests with good coverage, or manually tested (and explained how)
   - [ ] ๐Ÿ’ก Added comments for complex logic
   - [ ] ๐Ÿงพ Updated `CHANGELOG.md` (if needed)
   - [ ] ๐Ÿ“š Updated documentation in `site/content/in-dev/unreleased` (if needed)
   


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: HMS federation doesn't build due to outdated depdency [polaris]

Posted by "nandorKollar (via GitHub)" <gi...@apache.org>.
nandorKollar commented on PR #3795:
URL: https://github.com/apache/polaris/pull/3795#issuecomment-3914404910

   The build with HMS federation is currently failing with
   ```
   Caused by: java.lang.IllegalStateException: SRCFG00051: Could not generate ConfigMapping io.quarkiverse.amazon.rds.runtime.RdsBuildTimeConfig$$CMImpl
           at io.smallrye.config.ConfigMappingInterface.getClassBytes(ConfigMappingInterface.java:195)
           at io.smallrye.config.ConfigMappingLoader.loadClass(ConfigMappingLoader.java:177)
           at io.smallrye.config.ConfigMappingLoader.loadImplementation(ConfigMappingLoader.java:156)
           at io.smallrye.config.ConfigMappingLoader$1.computeValue(ConfigMappingLoader.java:29)
           at io.smallrye.config.ConfigMappingLoader$1.computeValue(ConfigMappingLoader.java:26)
           at io.smallrye.config.ConfigMappingLoader.configMappingSecrets(ConfigMappingLoader.java:91)
           at io.smallrye.config.ConfigMappings$ConfigClass.<init>(ConfigMappings.java:159)
           at io.smallrye.config.ConfigMappings$ConfigClass.configClass(ConfigMappings.java:220)
           at io.smallrye.config.ConfigMappings$ConfigClass.configClass(ConfigMappings.java:235)
           at io.quarkus.deployment.configuration.BuildTimeConfigurationReader.<init>(BuildTimeConfigurationReader.java:127)
           at io.quarkus.deployment.configuration.BuildTimeConfigurationReader.<init>(BuildTimeConfigurationReader.java:97)
           at io.quarkus.deployment.ExtensionLoader.loadStepsFrom(ExtensionLoader.java:144)
           at io.quarkus.deployment.QuarkusAugmentor.run(QuarkusAugmentor.java:109)
           at io.quarkus.runner.bootstrap.AugmentActionImpl.runAugment(AugmentActionImpl.java:385)
           at io.quarkus.runner.bootstrap.AugmentActionImpl.createProductionApplication(AugmentActionImpl.java:182)
           at io.quarkus.gradle.tasks.worker.BuildWorker.execute(BuildWorker.java:62)
           at org.gradle.workers.internal.DefaultWorkerServer.execute(DefaultWorkerServer.java:68)
           at org.gradle.workers.internal.AbstractClassLoaderWorker$1.create(AbstractClassLoaderWorker.java:54)
           at org.gradle.workers.internal.AbstractClassLoaderWorker$1.create(AbstractClassLoaderWorker.java:48)
           at org.gradle.internal.classloader.ClassLoaderUtils.executeInClassloader(ClassLoaderUtils.java:102)
           at org.gradle.workers.internal.AbstractClassLoaderWorker.executeInClassLoader(AbstractClassLoaderWorker.java:48)
           at org.gradle.workers.internal.IsolatedClassloaderWorker.run(IsolatedClassloaderWorker.java:49)
           at org.gradle.workers.internal.IsolatedClassloaderWorker.run(IsolatedClassloaderWorker.java:30)
           at org.gradle.workers.internal.WorkerDaemonServer.run(WorkerDaemonServer.java:109)
           at org.gradle.workers.internal.WorkerDaemonServer.run(WorkerDaemonServer.java:78)
           at org.gradle.process.internal.worker.request.WorkerAction.lambda$run$1(WorkerAction.java:150)
           at org.gradle.process.internal.worker.child.WorkerLogEventListener.withWorkerLoggingProtocol(WorkerLogEventListener.java:41)
           at org.gradle.process.internal.worker.request.WorkerAction.lambda$run$2(WorkerAction.java:150)
           at org.gradle.internal.operations.CurrentBuildOperationRef.with(CurrentBuildOperationRef.java:84)
           at org.gradle.process.internal.worker.request.WorkerAction.run(WorkerAction.java:142)
           at org.gradle.internal.dispatch.MethodInvocation.invokeOn(MethodInvocation.java:77)
           at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:28)
           at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:19)
           at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:182)
           at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:164)
           at org.gradle.internal.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:414)
           at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
           at org.gradle.internal.concurrent.AbstractManagedExecutor$1.run(AbstractManagedExecutor.java:47)
   Caused by: java.lang.IllegalArgumentException
           at org.objectweb.asm.MethodVisitor.<init>(Unknown Source)
           at io.smallrye.config.ConfigMappingGenerator$ObjectCreatorMethodVisitor.<init>(ConfigMappingGenerator.java:981)
           at io.smallrye.config.ConfigMappingGenerator.generate(ConfigMappingGenerator.java:138)
           at io.smallrye.config.ConfigMappingInterface.getClassBytes(ConfigMappingInterface.java:193)
           ... 37 more
   ```
   
   It seems that the problematic dependency is `asm-all`, which is a transitive dependency of Hive. I think that it used to work on 1.2.x, because the same module is a transitive dependency of other modules too, and the right version was shaded the wrong one on the classpath. On 1.3.x and latest master this shading is no longer there.
   Comparing the two dependency graphs for asm module:
   1.2.x:
   ```
   |    |    |    |    |         \--- org.ow2.asm:asm:9.7.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |         \--- asm:asm:3.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
        |    +--- org.ow2.asm:asm:9.8
        |    +--- org.ow2.asm:asm-commons:9.8
        |    |    +--- org.ow2.asm:asm:9.8
        |    |    \--- org.ow2.asm:asm-tree:9.8
        |    |         \--- org.ow2.asm:asm:9.8
        |    \--- org.ow2.asm:asm-tree:9.8 (*)
   |    |    |    |    |         \--- org.ow2.asm:asm:9.7.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |         \--- asm:asm:3.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |    |    |         \--- org.ow2.asm:asm:9.7.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |    |    |         \--- org.ow2.asm:asm:9.7.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   ```
   
   master:
   ```
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |         \--- asm:asm:3.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
        |    +--- org.ow2.asm:asm:9.8
        |    +--- org.ow2.asm:asm-commons:9.8
        |    |    +--- org.ow2.asm:asm:9.8
        |    |    \--- org.ow2.asm:asm-tree:9.8
        |    |         \--- org.ow2.asm:asm:9.8
        |    \--- org.ow2.asm:asm-tree:9.8 (*)
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |         \--- asm:asm:3.1
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   |    |    |    +--- org.ow2.asm:asm-all:5.0.2
   ```


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: HMS federation doesn't build due to outdated depdency [polaris]

Posted by "singhpk234 (via GitHub)" <gi...@apache.org>.
singhpk234 commented on PR #3795:
URL: https://github.com/apache/polaris/pull/3795#issuecomment-3961247371

   is it possible to add some testing around this @nandorKollar to make this future proof ? 


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org