[cleanup][transaction] Add Maven Modernizer plugin in pulsar-transact…

…ion (#16964)
apache · Aug 7, 2022 · edcf67a · edcf67a
1 parent 7a65f3f
commit edcf67a
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 2 deletions.
diff --git a/pulsar-transaction/common/pom.xml b/pulsar-transaction/common/pom.xml
@@ -37,6 +37,23 @@
     </dependencies>
     <build>
         <plugins>
+            <plugin>
+                <groupId>org.gaul</groupId>
+                <artifactId>modernizer-maven-plugin</artifactId>
+                <configuration>
+                    <failOnViolations>true</failOnViolations>
+                    <javaVersion>8</javaVersion>
+                </configuration>
+                <executions>
+                    <execution>
+                        <id>modernizer</id>
+                        <phase>verify</phase>
+                        <goals>
+                            <goal>modernizer</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
             <plugin>
                 <groupId>com.github.spotbugs</groupId>
                 <artifactId>spotbugs-maven-plugin</artifactId>

diff --git a/pulsar-transaction/coordinator/pom.xml b/pulsar-transaction/coordinator/pom.xml
@@ -64,6 +64,23 @@
 
     <build>
         <plugins>
+            <plugin>
+                <groupId>org.gaul</groupId>
+                <artifactId>modernizer-maven-plugin</artifactId>
+                <configuration>
+                    <failOnViolations>true</failOnViolations>
+                    <javaVersion>8</javaVersion>
+                </configuration>
+                <executions>
+                    <execution>
+                        <id>modernizer</id>
+                        <phase>verify</phase>
+                        <goals>
+                            <goal>modernizer</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>properties-maven-plugin</artifactId>

diff --git a/...ator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionSubscription.java b/...ator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionSubscription.java
@@ -18,7 +18,7 @@
  */
 package org.apache.pulsar.transaction.coordinator;
 
-import com.google.common.base.Objects;
+import java.util.Objects;
 import lombok.Builder;
 import lombok.Data;
 
@@ -34,7 +34,7 @@ public class TransactionSubscription implements Comparable<TransactionSubscripti
 
     @Override
     public int hashCode() {
-        return Objects.hashCode(topic, subscription);
+        return Objects.hash(topic, subscription);
     }
 
     @Override