Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/21697#discussion_r200186004
--- Diff: pom.xml ---
@@ -114,6 +114,7 @@
<maven.compiler.source>${java.version}</maven.compiler.source>
<maven.compiler.target>${java.version}</maven.compiler.target>
<maven.version>3.3.9</maven.version>
+ <maven.surefire.plugin.version>2.22.0</maven.surefire.plugin.version>
--- End diff --
The parent POM here already declares the surefire plugin, but version
2.20.1. That would need to be changed. If you do, you don't need a version in
the child POM, nor do you really need a property to control the version and the
extra indirection.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
