pan3793 commented on code in PR #48233:
URL: https://github.com/apache/spark/pull/48233#discussion_r1809765153


##########
project/SparkBuild.scala:
##########
@@ -954,7 +954,12 @@ object Unsafe {
 object DockerIntegrationTests {
   // This serves to override the override specified in DependencyOverrides:
   lazy val settings = Seq(
-    dependencyOverrides += "com.google.guava" % "guava" % "33.1.0-jre"
+    dependencyOverrides += {
+      val guavaVersion =
+        SbtPomKeys.effectivePom.value.getProperties.get(
+          "guava.version").asInstanceOf[String]
+      "com.google.guava" % "guava" % guavaVersion

Review Comment:
   Just out of curiosity, how will the Guava version be resolved without this 
override?



-- 
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]

Reply via email to