You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by pe...@apache.org on 2022/08/07 03:44:21 UTC

[pulsar] branch master updated: [cleanup][transaction] Add Maven Modernizer plugin in pulsar-transaction (#16964)

This is an automated email from the ASF dual-hosted git repository.

penghui pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new edcf67a9518 [cleanup][transaction] Add Maven Modernizer plugin in pulsar-transaction (#16964)
edcf67a9518 is described below

commit edcf67a9518888224abce2453bc16093eed0e093
Author: youzipi <bl...@qq.com>
AuthorDate: Sun Aug 7 11:44:14 2022 +0800

    [cleanup][transaction] Add Maven Modernizer plugin in pulsar-transaction (#16964)
---
 pulsar-transaction/common/pom.xml                       | 17 +++++++++++++++++
 pulsar-transaction/coordinator/pom.xml                  | 17 +++++++++++++++++
 .../coordinator/TransactionSubscription.java            |  4 ++--
 3 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/pulsar-transaction/common/pom.xml b/pulsar-transaction/common/pom.xml
index bde9aba1f75..be22c096316 100644
--- 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
index 2a96b3159fb..ccee4acb510 100644
--- 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/pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionSubscription.java b/pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionSubscription.java
index bcda38ea876..38d946fd54d 100644
--- a/pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionSubscription.java
+++ b/pulsar-transaction/coordinator/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