suxiaogang223 commented on code in PR #66321:
URL: https://github.com/apache/doris/pull/66321#discussion_r3711747250
##########
fe/be-java-extensions/java-udf/src/main/resources/package.xml:
##########
@@ -34,6 +34,19 @@ under the License.
<unpackOptions>
<excludes>
<exclude>**/Log4j2Plugins.dat</exclude>
+ <!--
+ TODO: Remove these Paimon exclusions after picking the
master change that
+ removes the hive-catalog-shade dependency.
+
+ hive-catalog-shade embeds Paimon 1.3 classes. This fat
jar is on the
+ JVM application classpath and therefore wins
parent-first resolution
+ over paimon-connector's isolated Paimon runtime. Java
UDF does not use
+ Paimon; keep those embedded classes out so the
connector owns its SDK.
+ -->
+ <exclude>org/apache/paimon/**</exclude>
+ <!-- ServiceLoader also consults parent resources. Do not
retain service
+ descriptors whose providers were removed with the
embedded SDK. -->
+ <exclude>META-INF/services/org.apache.paimon*</exclude>
Review Comment:
Fixed in 20fbee9a10. The Java UDF assembly now excludes
META-INF/services/java.time.chrono.Chronology together with the stripped shaded
Paimon providers. The full FE build and java-udf packaging completed
successfully, and the packaged jar was checked to contain neither the orphan
descriptor nor root Paimon classes.
--
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]
