conker84 commented on a change in pull request #3851: URL: https://github.com/apache/zeppelin/pull/3851#discussion_r456772892
########## 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: IMHO I prefer this approach because the Neo4j package is required only here, but if you think your is better as in the whole project is already adopted it's fine with me. Let me know how I should proceed. ---------------------------------------------------------------- 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