sekikn commented on code in PR #653:
URL: https://github.com/apache/incubator-sedona/pull/653#discussion_r935085976
##########
pom.xml:
##########
@@ -66,6 +66,12 @@
</properties>
<dependencies>
+ <dependency>
+ <groupId>com.google.guava</groupId>
+ <artifactId>guava</artifactId>
+ <version>${guava.version}</version>
+ <scope>${dependency.scope}</scope>
Review Comment:
> Why does guava need to be added, doesn't Hadoop have this already?
Yes it should do, but for some reason, wrong version of Guava is used for
unit test and it causes NoSuchMethodError. Some other libraries that have a
transitive dependency on Guava seem to cause this problem. For resolving it,
I'll change the dependency declaration order in pom.xml, instead of adding this
dependency.
--
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]
