LuciferYang commented on code in PR #43436:
URL: https://github.com/apache/spark/pull/43436#discussion_r1426350579
##########
connector/connect/client/jvm/pom.xml:
##########
@@ -124,6 +124,10 @@
<include>io.grpc.**</include>
</includes>
</relocation>
+ <relocation>
Review Comment:
> The current Apache Spark includes two versions of Guava, which are (The
historical reasons are not clear to me. @LuciferYang may know some background):
Sorry I missed this ping. 14.0.1 is because Hive 2.3.9 depends on Guava
14.0.1, which is shaded in the `network-common` module and used by other
non-connect modules. `32.0.1-jre` is dedicated to Connect-related modules.
Currently, the connect server and connect jvm client modules should have shaded
it respectively, and for the Guava 32.0.1-jre, the dependency
`com.google.guava:failureaccess` should also be shaded.
--
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]
