This is an automated email from the ASF dual-hosted git repository.
jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 14b77747c7 [Dependency] Cleanup maven plugin versions (#13878)
14b77747c7 is described below
commit 14b77747c7e4a11b083eccdb6c35d4a6055a5e92
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Fri Aug 23 11:39:40 2024 -0700
[Dependency] Cleanup maven plugin versions (#13878)
---
pinot-connectors/pinot-spark-2-connector/pom.xml | 15 ---
pinot-connectors/pinot-spark-3-connector/pom.xml | 15 ---
pinot-connectors/pinot-spark-common/pom.xml | 22 -----
pinot-controller/pom.xml | 1 -
pinot-distribution/pom.xml | 2 -
.../pinot-input-format/pinot-protobuf/pom.xml | 2 -
pinot-tools/pom.xml | 2 +-
pom.xml | 109 +++++++++++----------
8 files changed, 56 insertions(+), 112 deletions(-)
diff --git a/pinot-connectors/pinot-spark-2-connector/pom.xml
b/pinot-connectors/pinot-spark-2-connector/pom.xml
index a40d20dfa8..fde5cf1af2 100644
--- a/pinot-connectors/pinot-spark-2-connector/pom.xml
+++ b/pinot-connectors/pinot-spark-2-connector/pom.xml
@@ -122,7 +122,6 @@
Thus, explicitly adding this plugin to a new profile to sign the
files at the end all at once. -->
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-gpg-plugin</artifactId>
- <version>3.2.5</version>
</plugin>
</plugins>
</build>
@@ -155,20 +154,6 @@
<plugin>
<groupId>org.scalatest</groupId>
<artifactId>scalatest-maven-plugin</artifactId>
- <version>2.2.0</version>
- <configuration>
-
<reportsDirectory>${project.build.directory}/surefire-reports</reportsDirectory>
- <junitxml>.</junitxml>
- <parallel>false</parallel>
- </configuration>
- <executions>
- <execution>
- <id>test</id>
- <goals>
- <goal>test</goal>
- </goals>
- </execution>
- </executions>
</plugin>
</plugins>
</build>
diff --git a/pinot-connectors/pinot-spark-3-connector/pom.xml
b/pinot-connectors/pinot-spark-3-connector/pom.xml
index 5b16c720ee..1c3661c723 100644
--- a/pinot-connectors/pinot-spark-3-connector/pom.xml
+++ b/pinot-connectors/pinot-spark-3-connector/pom.xml
@@ -106,7 +106,6 @@
Thus, explicitly adding this plugin to a new profile to sign the
files at the end all at once. -->
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-gpg-plugin</artifactId>
- <version>3.2.5</version>
</plugin>
</plugins>
</build>
@@ -139,20 +138,6 @@
<plugin>
<groupId>org.scalatest</groupId>
<artifactId>scalatest-maven-plugin</artifactId>
- <version>2.2.0</version>
- <configuration>
-
<reportsDirectory>${project.build.directory}/surefire-reports</reportsDirectory>
- <junitxml>.</junitxml>
- <parallel>false</parallel>
- </configuration>
- <executions>
- <execution>
- <id>test</id>
- <goals>
- <goal>test</goal>
- </goals>
- </execution>
- </executions>
</plugin>
</plugins>
</build>
diff --git a/pinot-connectors/pinot-spark-common/pom.xml
b/pinot-connectors/pinot-spark-common/pom.xml
index e69c6a0a93..942f8c97e4 100644
--- a/pinot-connectors/pinot-spark-common/pom.xml
+++ b/pinot-connectors/pinot-spark-common/pom.xml
@@ -78,16 +78,9 @@
</dependencies>
<build>
<plugins>
- <!-- scala build -->
<plugin>
<groupId>org.xolstice.maven.plugins</groupId>
<artifactId>protobuf-maven-plugin</artifactId>
- <version>0.6.1</version>
- <configuration>
-
<protocArtifact>com.google.protobuf:protoc:3.19.2:exe:${os.detected.classifier}</protocArtifact>
- <pluginId>grpc-java</pluginId>
-
<pluginArtifact>io.grpc:protoc-gen-grpc-java:1.44.1:exe:${os.detected.classifier}</pluginArtifact>
- </configuration>
<executions>
<execution>
<goals>
@@ -114,7 +107,6 @@
Thus, explicitly adding this plugin to a new profile to sign the
files at the end all at once. -->
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-gpg-plugin</artifactId>
- <version>3.2.5</version>
</plugin>
</plugins>
</build>
@@ -147,20 +139,6 @@
<plugin>
<groupId>org.scalatest</groupId>
<artifactId>scalatest-maven-plugin</artifactId>
- <version>2.2.0</version>
- <configuration>
-
<reportsDirectory>${project.build.directory}/surefire-reports</reportsDirectory>
- <junitxml>.</junitxml>
- <parallel>false</parallel>
- </configuration>
- <executions>
- <execution>
- <id>test</id>
- <goals>
- <goal>test</goal>
- </goals>
- </execution>
- </executions>
</plugin>
</plugins>
</build>
diff --git a/pinot-controller/pom.xml b/pinot-controller/pom.xml
index ef2c82378a..466b5d9c92 100644
--- a/pinot-controller/pom.xml
+++ b/pinot-controller/pom.xml
@@ -103,7 +103,6 @@
<plugin>
<groupId>com.github.eirslett</groupId>
<artifactId>frontend-maven-plugin</artifactId>
- <version>1.15.0</version>
<!-- optional -->
<configuration>
diff --git a/pinot-distribution/pom.xml b/pinot-distribution/pom.xml
index 9beb154540..f84aa8ef27 100644
--- a/pinot-distribution/pom.xml
+++ b/pinot-distribution/pom.xml
@@ -50,7 +50,6 @@
<plugin>
<groupId>net.nicoulaj.maven.plugins</groupId>
<artifactId>checksum-maven-plugin</artifactId>
- <version>1.11</version>
<executions>
<execution>
<goals>
@@ -132,7 +131,6 @@
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>exec-maven-plugin</artifactId>
- <version>3.4.1</version>
<executions>
<execution>
<id>remove-build-directory</id>
diff --git a/pinot-plugins/pinot-input-format/pinot-protobuf/pom.xml
b/pinot-plugins/pinot-input-format/pinot-protobuf/pom.xml
index 14b0ab5a11..19afe9ff35 100644
--- a/pinot-plugins/pinot-input-format/pinot-protobuf/pom.xml
+++ b/pinot-plugins/pinot-input-format/pinot-protobuf/pom.xml
@@ -95,9 +95,7 @@
<plugin>
<groupId>org.xolstice.maven.plugins</groupId>
<artifactId>protobuf-maven-plugin</artifactId>
- <version>0.6.1</version>
<configuration>
-
<protocArtifact>com.google.protobuf:protoc:${protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<protoTestSourceRoot>${basedir}/src/test/resources</protoTestSourceRoot>
</configuration>
<executions>
diff --git a/pinot-tools/pom.xml b/pinot-tools/pom.xml
index ec41e88232..b40fac3108 100644
--- a/pinot-tools/pom.xml
+++ b/pinot-tools/pom.xml
@@ -452,8 +452,8 @@
</executions>
</plugin>
<plugin>
+ <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-resources-plugin</artifactId>
- <version>3.3.1</version>
<executions>
<execution>
<id>copy-kafka-resources</id>
diff --git a/pom.xml b/pom.xml
index 157abe03b1..819dd4e9a1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -126,8 +126,6 @@
<!-- Sets the VM argument line used when unit tests are run. -->
<argLine>-Xms4g -Xmx4g</argLine>
<SKIP_INTEGRATION_TESTS>true</SKIP_INTEGRATION_TESTS>
- <surefire.version>3.4.0</surefire.version>
- <maven-enforcer-plugin.version>3.5.0</maven-enforcer-plugin.version>
<!-- Checkstyle violation prop.-->
<checkstyle.violation.severity>warning</checkstyle.violation.severity>
@@ -135,8 +133,6 @@
<!-- Configuration for Packaging -->
<shade.prefix>org.apache.pinot.shaded</shade.prefix>
- <maven-jar-plugin.version>3.4.2</maven-jar-plugin.version>
- <maven-shade-plugin.version>3.6.0</maven-shade-plugin.version>
<shade.phase.prop>none</shade.phase.prop>
<avro.version>1.11.3</avro.version>
@@ -1800,6 +1796,7 @@
<build>
<defaultGoal>clean install</defaultGoal>
+
<extensions>
<extension>
<groupId>kr.motd.maven</groupId>
@@ -1807,6 +1804,7 @@
<version>1.7.1</version>
</extension>
</extensions>
+
<pluginManagement>
<plugins>
<plugin>
@@ -1867,30 +1865,20 @@
</execution>
</executions>
</plugin>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-assembly-plugin</artifactId>
- <version>3.7.1</version>
- </plugin>
- <plugin>
- <artifactId>maven-resources-plugin</artifactId>
- <version>3.3.1</version>
- </plugin>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-dependency-plugin</artifactId>
- <version>3.7.1</version>
- </plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
- <version>3.8.0</version>
<configuration>
<!-- Remove this after fixing all javadoc warnings -->
<doclint>none</doclint>
<source>${jdk.version}</source>
</configuration>
</plugin>
+ <plugin>
+ <groupId>org.jacoco</groupId>
+ <artifactId>jacoco-maven-plugin</artifactId>
+ <version>0.8.12</version>
+ </plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-source-plugin</artifactId>
@@ -1903,19 +1891,9 @@
</execution>
</executions>
</plugin>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-eclipse-plugin</artifactId>
- <version>2.10</version>
- <configuration>
- <downloadSources>true</downloadSources>
- <downloadJavadocs>true</downloadJavadocs>
- </configuration>
- </plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
- <version>${maven-jar-plugin.version}</version>
<executions>
<execution>
<goals>
@@ -1939,7 +1917,6 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
- <version>${surefire.version}</version>
<configuration>
<forkCount>1</forkCount>
<reuseForks>false</reuseForks>
@@ -1974,8 +1951,6 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
- <!-- Note that 3.0.0, while newer, has bugs w/provided transitive
dependencies -->
- <version>${maven-enforcer-plugin.version}</version>
<executions>
<execution>
<id>enforce-dependency-convergence</id>
@@ -2024,10 +1999,19 @@
<artifactId>license-maven-plugin</artifactId>
<version>4.5</version>
</plugin>
+ <plugin>
+ <groupId>org.codehaus.mojo</groupId>
+ <artifactId>buildnumber-maven-plugin</artifactId>
+ <version>3.2.0</version>
+ </plugin>
+ <plugin>
+ <groupId>org.codehaus.mojo</groupId>
+ <artifactId>versions-maven-plugin</artifactId>
+ <version>2.17.1</version>
+ </plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-release-plugin</artifactId>
- <version>3.1.1</version>
<dependencies>
<dependency>
<groupId>org.apache.maven.scm</groupId>
@@ -2044,8 +2028,7 @@
<artifactId>protobuf-maven-plugin</artifactId>
<version>0.6.1</version>
<configuration>
-
<protocArtifact>com.google.protobuf:protoc:${protobuf.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<pluginId>grpc-java</pluginId>
<pluginArtifact>io.grpc:protoc-gen-grpc-java:${grpc.version}:exe:${os.detected.classifier}</pluginArtifact>
</configuration>
@@ -2063,7 +2046,6 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-compiler-plugin</artifactId>
- <version>3.13.0</version>
<configuration>
<release>${jdk.version}</release>
<source>${jdk.version}</source>
@@ -2072,11 +2054,6 @@
<encoding>${project.build.sourceEncoding}</encoding>
</configuration>
</plugin>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-remote-resources-plugin</artifactId>
- <version>3.2.0</version>
- </plugin>
<plugin>
<groupId>com.googlecode.fmpp-maven-plugin</groupId>
<artifactId>fmpp-maven-plugin</artifactId>
@@ -2151,13 +2128,46 @@
</javacArgs>
</configuration>
</plugin>
+ <plugin>
+ <groupId>org.scalatest</groupId>
+ <artifactId>scalatest-maven-plugin</artifactId>
+ <version>2.2.0</version>
+ <configuration>
+
<reportsDirectory>${project.build.directory}/surefire-reports</reportsDirectory>
+ <junitxml>.</junitxml>
+ <parallel>false</parallel>
+ </configuration>
+ <executions>
+ <execution>
+ <id>test</id>
+ <goals>
+ <goal>test</goal>
+ </goals>
+ </execution>
+ </executions>
+ </plugin>
+ <plugin>
+ <groupId>com.github.eirslett</groupId>
+ <artifactId>frontend-maven-plugin</artifactId>
+ <version>1.15.0</version>
+ </plugin>
+ <plugin>
+ <groupId>net.nicoulaj.maven.plugins</groupId>
+ <artifactId>checksum-maven-plugin</artifactId>
+ <version>1.11</version>
+ </plugin>
+ <plugin>
+ <groupId>org.codehaus.mojo</groupId>
+ <artifactId>exec-maven-plugin</artifactId>
+ <version>3.4.1</version>
+ </plugin>
</plugins>
</pluginManagement>
+
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
- <version>3.4.0</version>
<configuration>
<configLocation>${pinot.root}/config/checkstyle.xml</configLocation>
<suppressionsLocation>${pinot.root}/config/suppressions.xml</suppressionsLocation>
@@ -2248,7 +2258,6 @@
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
- <version>0.8.12</version>
<executions>
<execution>
<goals>
@@ -2273,11 +2282,6 @@
</includes>
</configuration>
</plugin>
- <plugin>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>sonar-maven-plugin</artifactId>
- <version>2.7.1</version>
- </plugin>
<plugin>
<groupId>com.diffplug.spotless</groupId>
<artifactId>spotless-maven-plugin</artifactId>
@@ -2285,7 +2289,6 @@
<plugin>
<groupId>com.mycila</groupId>
<artifactId>license-maven-plugin</artifactId>
- <version>4.5</version>
<configuration>
<licenseSets>
<licenseSet>
@@ -2405,7 +2408,6 @@
<plugin>
<groupId>org.apache.rat</groupId>
<artifactId>apache-rat-plugin</artifactId>
- <version>0.16.1</version>
<executions>
<execution>
<phase>verify</phase>
@@ -2502,7 +2504,6 @@
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>buildnumber-maven-plugin</artifactId>
- <version>3.2.0</version>
<executions>
<execution>
<phase>validate</phase>
@@ -2515,11 +2516,10 @@
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>versions-maven-plugin</artifactId>
- <version>2.17.1</version>
</plugin>
<plugin>
+ <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
- <version>${maven-shade-plugin.version}</version>
<dependencies>
<dependency>
<groupId>org.apache.logging.log4j</groupId>
@@ -2601,6 +2601,8 @@
</plugin>
</plugins>
</build>
+
+ <!-- TODO: Revisit if we need reporting -->
<reporting>
<plugins>
<plugin>
@@ -2614,7 +2616,6 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-project-info-reports-plugin</artifactId>
- <version>3.6.2</version>
<configuration>
<dependencyLocationsEnabled>false</dependencyLocationsEnabled>
</configuration>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]