zhangfengcdt commented on code in PR #2971:
URL: https://github.com/apache/parquet-java/pull/2971#discussion_r1723553839
##########
parquet-hadoop/pom.xml:
##########
@@ -203,6 +209,12 @@
<version>${commons-lang3.version}</version>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.locationtech.jts</groupId>
+ <artifactId>jts-core</artifactId>
+ <version>${jts.version}</version>
+ <scope>test</scope>
Review Comment:
Only test scoped jts-core is under the parquet-hadoop pom, the other one
(w/o test scope) is under the parquet-column pom. If i removed the test scope,
it errors out:
```
[ERROR] Non-test scoped test only dependencies found:
[ERROR] org.locationtech.jts:jts-core:jar:1.19.0:compile
```
I think it suggests that the jts-core dependency is included in the compile
scope in parquet-column, but it's being used in test code here in
parquet-hadoop.
--
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]
