This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new 2fc9c6e5f1 HDDS-12034. Enable sortpom in hdds-interface-admin, -client
and -server. (#7660)
2fc9c6e5f1 is described below
commit 2fc9c6e5f16724ef4bdc4ba4f9e1988e959a25cf
Author: Nandakumar Vadivelu <[email protected]>
AuthorDate: Wed Jan 8 10:03:33 2025 +0530
HDDS-12034. Enable sortpom in hdds-interface-admin, -client and -server.
(#7660)
---
hadoop-hdds/interface-admin/pom.xml | 21 ++++------
hadoop-hdds/interface-client/pom.xml | 78 ++++++++++++------------------------
hadoop-hdds/interface-server/pom.xml | 58 +++++++++------------------
3 files changed, 53 insertions(+), 104 deletions(-)
diff --git a/hadoop-hdds/interface-admin/pom.xml
b/hadoop-hdds/interface-admin/pom.xml
index 9412242308..047db244fa 100644
--- a/hadoop-hdds/interface-admin/pom.xml
+++ b/hadoop-hdds/interface-admin/pom.xml
@@ -12,10 +12,7 @@
See the License for the specific language governing permissions and
limitations under the License. See accompanying LICENSE file.
-->
-<project xmlns="http://maven.apache.org/POM/4.0.0"
- xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
- xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
-https://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
https://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<parent>
<groupId>org.apache.ozone</groupId>
@@ -24,15 +21,15 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
</parent>
<artifactId>hdds-interface-admin</artifactId>
<version>2.0.0-SNAPSHOT</version>
- <description>Apache Ozone Distributed Data Store Admin interface
- </description>
- <name>Apache Ozone HDDS Admin Interface</name>
<packaging>jar</packaging>
+ <name>Apache Ozone HDDS Admin Interface</name>
+ <description>Apache Ozone Distributed Data Store Admin
interface</description>
<properties>
- <maven.test.skip>true</maven.test.skip> <!-- no testable code in this
module -->
- <spotbugs.skip>true</spotbugs.skip> <!-- only generated code in this
module -->
- <sort.skip>true</sort.skip>
+ <!-- no testable code in this module -->
+ <maven.test.skip>true</maven.test.skip>
+ <!-- only generated code in this module -->
+ <spotbugs.skip>true</spotbugs.skip>
</properties>
<dependencies>
@@ -72,9 +69,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<goal>test-compile</goal>
</goals>
<configuration>
- <protocArtifact>
-
com.google.protobuf:protoc:${proto2.hadooprpc.protobuf.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${proto2.hadooprpc.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/proto/</protoSourceRoot>
<outputDirectory>target/generated-sources/java</outputDirectory>
<clearOutputDirectory>false</clearOutputDirectory>
diff --git a/hadoop-hdds/interface-client/pom.xml
b/hadoop-hdds/interface-client/pom.xml
index b373d11d50..da6dec5cda 100644
--- a/hadoop-hdds/interface-client/pom.xml
+++ b/hadoop-hdds/interface-client/pom.xml
@@ -12,10 +12,7 @@
See the License for the specific language governing permissions and
limitations under the License. See accompanying LICENSE file.
-->
-<project xmlns="http://maven.apache.org/POM/4.0.0"
- xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
- xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
-https://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
https://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<parent>
<groupId>org.apache.ozone</groupId>
@@ -24,15 +21,15 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
</parent>
<artifactId>hdds-interface-client</artifactId>
<version>2.0.0-SNAPSHOT</version>
- <description>Apache Ozone Distributed Data Store Client interface
- </description>
- <name>Apache Ozone HDDS Client Interface</name>
<packaging>jar</packaging>
+ <name>Apache Ozone HDDS Client Interface</name>
+ <description>Apache Ozone Distributed Data Store Client
interface</description>
<properties>
- <maven.test.skip>true</maven.test.skip> <!-- no testable code in this
module -->
- <spotbugs.skip>true</spotbugs.skip> <!-- only generated code in this
module -->
- <sort.skip>true</sort.skip>
+ <!-- no testable code in this module -->
+ <maven.test.skip>true</maven.test.skip>
+ <!-- only generated code in this module -->
+ <spotbugs.skip>true</spotbugs.skip>
</properties>
<dependencies>
@@ -40,6 +37,10 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<groupId>com.google.protobuf</groupId>
<artifactId>protobuf-java</artifactId>
</dependency>
+ <dependency>
+ <groupId>javax.annotation</groupId>
+ <artifactId>javax.annotation-api</artifactId>
+ </dependency>
<dependency>
<groupId>org.apache.hadoop.thirdparty</groupId>
<artifactId>hadoop-shaded-protobuf_3_25</artifactId>
@@ -49,10 +50,6 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<artifactId>ratis-thirdparty-misc</artifactId>
<version>${ratis.thirdparty.version}</version>
</dependency>
- <dependency>
- <groupId>javax.annotation</groupId>
- <artifactId>javax.annotation-api</artifactId>
- </dependency>
</dependencies>
<build>
<plugins>
@@ -82,9 +79,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<goal>test-compile-custom</goal>
</goals>
<configuration>
- <protocArtifact>
-
com.google.protobuf:protoc:${grpc.protobuf-compile.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${grpc.protobuf-compile.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/proto/</protoSourceRoot>
<includes>
<include>DatanodeClientProtocol.proto</include>
@@ -93,9 +88,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<outputDirectory>target/generated-sources/java</outputDirectory>
<clearOutputDirectory>false</clearOutputDirectory>
<pluginId>grpc-java</pluginId>
- <pluginArtifact>
-
io.grpc:protoc-gen-grpc-java:${io.grpc.version}:exe:${os.detected.classifier}
- </pluginArtifact>
+
<pluginArtifact>io.grpc:protoc-gen-grpc-java:${io.grpc.version}:exe:${os.detected.classifier}</pluginArtifact>
</configuration>
</execution>
<execution>
@@ -105,9 +98,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<goal>test-compile</goal>
</goals>
<configuration>
- <protocArtifact>
-
com.google.protobuf:protoc:${proto2.hadooprpc.protobuf.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${proto2.hadooprpc.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/proto/</protoSourceRoot>
<includes>
<include>hdds.proto</include>
@@ -124,9 +115,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<goal>test-compile</goal>
</goals>
<configuration>
- <protocArtifact>
-
com.google.protobuf:protoc:${proto3.hadooprpc.protobuf.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${proto3.hadooprpc.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/proto/</protoSourceRoot>
<includes>
<include>hdds.proto</include>
@@ -143,38 +132,21 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<version>${maven-antrun-plugin.version}</version>
<executions>
<execution>
+ <goals>
+ <goal>run</goal>
+ </goals>
<phase>generate-sources</phase>
<configuration>
<target>
- <replace token="com.google.protobuf"
-
value="org.apache.ratis.thirdparty.com.google.protobuf"
-
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/datanode/proto">
- </replace>
- <replace token="io.grpc"
- value="org.apache.ratis.thirdparty.io.grpc"
-
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/datanode/proto">
- </replace>
- <replace token="com.google.common"
- value="org.apache.ratis.thirdparty.com.google.common"
-
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/datanode/proto">
- </replace>
- <replace token="org.apache.hadoop.hdds.protocol.proto"
- value="org.apache.hadoop.hdds.protocol.proto3"
-
dir="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto">
- </replace>
- <replace token="com.google.protobuf"
- value="org.apache.hadoop.thirdparty.protobuf"
-
dir="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto">
- </replace>
- <move
file="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto"
-
tofile="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto3"/>
- <move file="target/generated-sources/java/proto3"
- tofile="target/generated-sources/java"/>
+ <replace
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/datanode/proto"
token="com.google.protobuf"
value="org.apache.ratis.thirdparty.com.google.protobuf" />
+ <replace
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/datanode/proto"
token="io.grpc" value="org.apache.ratis.thirdparty.io.grpc" />
+ <replace
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/datanode/proto"
token="com.google.common"
value="org.apache.ratis.thirdparty.com.google.common" />
+ <replace
dir="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto"
token="org.apache.hadoop.hdds.protocol.proto"
value="org.apache.hadoop.hdds.protocol.proto3" />
+ <replace
dir="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto"
token="com.google.protobuf" value="org.apache.hadoop.thirdparty.protobuf" />
+ <move
file="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto"
tofile="target/generated-sources/java/proto3/org/apache/hadoop/hdds/protocol/proto3"
/>
+ <move file="target/generated-sources/java/proto3"
tofile="target/generated-sources/java" />
</target>
</configuration>
- <goals>
- <goal>run</goal>
- </goals>
</execution>
</executions>
</plugin>
diff --git a/hadoop-hdds/interface-server/pom.xml
b/hadoop-hdds/interface-server/pom.xml
index 539a0a5430..83aa5f72e3 100644
--- a/hadoop-hdds/interface-server/pom.xml
+++ b/hadoop-hdds/interface-server/pom.xml
@@ -12,10 +12,7 @@
See the License for the specific language governing permissions and
limitations under the License. See accompanying LICENSE file.
-->
-<project xmlns="http://maven.apache.org/POM/4.0.0"
- xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
- xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
-https://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
https://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<parent>
<groupId>org.apache.ozone</groupId>
@@ -24,22 +21,18 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
</parent>
<artifactId>hdds-interface-server</artifactId>
<version>2.0.0-SNAPSHOT</version>
- <description>Apache Ozone Distributed Data Store Server interface
- </description>
- <name>Apache Ozone HDDS Server Interface</name>
<packaging>jar</packaging>
+ <name>Apache Ozone HDDS Server Interface</name>
+ <description>Apache Ozone Distributed Data Store Server
interface</description>
<properties>
- <maven.test.skip>true</maven.test.skip> <!-- no testable code in this
module -->
- <spotbugs.skip>true</spotbugs.skip> <!-- only generated code in this
module -->
- <sort.skip>true</sort.skip>
+ <!-- no testable code in this module -->
+ <maven.test.skip>true</maven.test.skip>
+ <!-- only generated code in this module -->
+ <spotbugs.skip>true</spotbugs.skip>
</properties>
<dependencies>
- <dependency>
- <groupId>org.apache.ratis</groupId>
- <artifactId>ratis-thirdparty-misc</artifactId>
- </dependency>
<dependency>
<groupId>com.google.protobuf</groupId>
<artifactId>protobuf-java</artifactId>
@@ -50,6 +43,10 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<groupId>org.apache.ozone</groupId>
<artifactId>hdds-interface-client</artifactId>
</dependency>
+ <dependency>
+ <groupId>org.apache.ratis</groupId>
+ <artifactId>ratis-thirdparty-misc</artifactId>
+ </dependency>
</dependencies>
<build>
<plugins>
@@ -79,9 +76,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<goal>test-compile-custom</goal>
</goals>
<configuration>
- <protocArtifact>
-
com.google.protobuf:protoc:${grpc.protobuf-compile.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${grpc.protobuf-compile.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/proto/</protoSourceRoot>
<includes>
<include>InterSCMProtocol.proto</include>
@@ -90,9 +85,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<outputDirectory>target/generated-sources/java</outputDirectory>
<clearOutputDirectory>false</clearOutputDirectory>
<pluginId>grpc-java</pluginId>
- <pluginArtifact>
-
io.grpc:protoc-gen-grpc-java:${io.grpc.version}:exe:${os.detected.classifier}
- </pluginArtifact>
+
<pluginArtifact>io.grpc:protoc-gen-grpc-java:${io.grpc.version}:exe:${os.detected.classifier}</pluginArtifact>
</configuration>
</execution>
<execution>
@@ -102,9 +95,7 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<goal>test-compile</goal>
</goals>
<configuration>
- <protocArtifact>
-
com.google.protobuf:protoc:${proto2.hadooprpc.protobuf.version}:exe:${os.detected.classifier}
- </protocArtifact>
+
<protocArtifact>com.google.protobuf:protoc:${proto2.hadooprpc.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
<protoSourceRoot>${basedir}/src/main/proto/</protoSourceRoot>
<excludes>
<exclude>InterSCMProtocol.proto</exclude>
@@ -121,26 +112,17 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd">
<version>${maven-antrun-plugin.version}</version>
<executions>
<execution>
+ <goals>
+ <goal>run</goal>
+ </goals>
<phase>generate-sources</phase>
<configuration>
<target>
- <replace token="com.google.protobuf"
-
value="org.apache.ratis.thirdparty.com.google.protobuf"
-
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/scm/proto">
- </replace>
- <replace token="io.grpc"
- value="org.apache.ratis.thirdparty.io.grpc"
-
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/scm/proto">
- </replace>
- <replace token="com.google.common"
- value="org.apache.ratis.thirdparty.com.google.common"
-
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/scm/proto">
- </replace>
+ <replace
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/scm/proto"
token="com.google.protobuf"
value="org.apache.ratis.thirdparty.com.google.protobuf" />
+ <replace
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/scm/proto"
token="io.grpc" value="org.apache.ratis.thirdparty.io.grpc" />
+ <replace
dir="target/generated-sources/java/org/apache/hadoop/hdds/protocol/scm/proto"
token="com.google.common" value="org.apache.ratis.thirdparty.com.google.common"
/>
</target>
</configuration>
- <goals>
- <goal>run</goal>
- </goals>
</execution>
</executions>
</plugin>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]