jongyoul commented on a change in pull request #3851:
URL: https://github.com/apache/zeppelin/pull/3851#discussion_r455071130
##########
File path: neo4j/pom.xml
##########
@@ -65,10 +63,9 @@
</dependency>
<dependency>
- <groupId>org.neo4j.test</groupId>
- <artifactId>neo4j-harness</artifactId>
- <version>${neo4j.version}</version>
- <scope>test</scope>
+ <groupId>org.testcontainers</groupId>
+ <artifactId>neo4j</artifactId>
+ <version>${testcontainers.version}</version>
Review comment:
Oh, I've missed it. BTW, I think we'd better define neo4j version in the
parent pom.xml as well and use it on this pom.xml without version information.
I remembered that we already had these kinds of ways for others even if it's
used in a single pom.xml only. WDYT?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
