update maven plugin check & format (#66)

.github/workflows/maven_build.yml → .github/workflows/maven_build_master.yml

-name: Maven Build
+name: Maven Build on Master Branch
 
 on:
-  pull_request:
+  push:
     branches:
       - master
 

.github/workflows/maven_build_pr.yml

+name: Maven Build on Pull Request
+
+on:
+  pull_request:
+    paths-ignore:
+      - 'docs/**'
+      - '**.md'
+      - '.*'
+
+jobs:
+  build:
+    name: Maven Build
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up JDK 8
+        uses: actions/setup-java@v2
+        with:
+          java-version: '8'
+          distribution: 'zulu'
+      - name: Build with Maven
+        run: mvn clean install

.github/workflows/maven_publish_snapshot.yml

-name: Maven Publish Snapshot
-
-on:
-  push:
-    branches:
-      - master
-    paths-ignore:
-      - '.github/**'
-      - 'docs/**'
-      - 'scripts/**'
-      - '**.md'
-      - '.*'
-
-jobs:
-  publish-snapshot:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - name: Setup Apache Maven Central
-        uses: actions/setup-java@v2
-        with:
-          java-version: '8'
-          distribution: 'zulu'
-          server-id: sonatype-nexus-snapshots
-          server-username: MAVEN_USERNAME
-          server-password: MAVEN_PASSWORD
-          gpg-passphrase: GPG_PASSPHRASE
-          gpg-private-key: ${{ secrets.GPG_PRIVATE_KEY }}
-      - name: Publish to Apache Maven Central
-        run: mvn clean verify gpg:sign install:install deploy:deploy
-        env:
-          MAVEN_USERNAME: ${{ secrets.OSSRH_USERNAME }}
-          MAVEN_PASSWORD: ${{ secrets.OSSRH_PASSWORD }}
-          GPG_PASSPHRASE: ${{ secrets.GPG_PASSPHRASE }}

README.md

@@ -3,7 +3,7 @@ OceanBase Log Client
 
 English | [简体中文](README_CN.md)
 
-[![Build Status](https://github.com/oceanbase/oblogclient/actions/workflows/maven_publish_snapshot.yml/badge.svg?branch=master)](https://github.com/oceanbase/oblogclient/actions/workflows/maven_publish_snapshot.yml)
+[![Build Status](https://github.com/oceanbase/oblogclient/actions/workflows/maven_build_master.yml/badge.svg?branch=master)](https://github.com/oceanbase/oblogclient/actions/workflows/maven_build_master.yml)
 [![Release](https://img.shields.io/github/release/oceanbase/oblogclient.svg)](https://github.com/oceanbase/oblogclient/releases)
 [![License](https://img.shields.io/badge/license-Mulan%20PSL%20v2-green.svg)](LICENSE)
 

README_CN.md

@@ -3,7 +3,7 @@ OceanBase 日志客户端
 
 [English](README.md) | 简体中文
 
-[![Build Status](https://github.com/oceanbase/oblogclient/actions/workflows/maven_publish_snapshot.yml/badge.svg?branch=master)](https://github.com/oceanbase/oblogclient/actions/workflows/maven_publish_snapshot.yml)
+[![Build Status](https://github.com/oceanbase/oblogclient/actions/workflows/maven_build_master.yml/badge.svg?branch=master)](https://github.com/oceanbase/oblogclient/actions/workflows/maven_build_master.yml)
 [![Release](https://img.shields.io/github/release/oceanbase/oblogclient.svg)](https://github.com/oceanbase/oblogclient/releases)
 [![License](https://img.shields.io/badge/license-Mulan%20PSL%20v2-green.svg)](LICENSE)
 

oblogclient-logproxy/src/main/java/com/oceanbase/clogproxy/client/connection/ClientHandlerV01.java

@@ -31,7 +31,7 @@ import org.apache.commons.lang3.Conversion;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-/* Compatitable for legacy V0 and V1 only, however you should not use */
+/** Compatible for legacy V0 and V1 only, however you should not use */
 public class ClientHandlerV01 {
 
     private static final Logger logger = LoggerFactory.getLogger(ClientHandlerV01.class);

oblogclient-logproxy/src/main/java/com/oceanbase/clogproxy/client/exception/LogProxyClientException.java

@@ -10,6 +10,7 @@ See the Mulan PSL v2 for more details. */
 
 package com.oceanbase.clogproxy.client.exception;
 
+
 import com.oceanbase.clogproxy.client.enums.ErrorCode;
 
 /**
@@ -93,7 +94,7 @@ public class LogProxyClientException extends RuntimeException {
      * @param needStop The flag of whether the client should stop the stream.
      */
     public LogProxyClientException(
-        ErrorCode code, String message, Throwable throwable, boolean needStop) {
+            ErrorCode code, String message, Throwable throwable, boolean needStop) {
         super(message, throwable);
         this.code = code;
         this.needStop = needStop;

pom.xml

@@ -188,6 +188,9 @@ See the Mulan PSL v2 for more details.
             <goals>
               <goal>jar</goal>
             </goals>
+            <configuration>
+              <failOnWarnings>true</failOnWarnings>
+            </configuration>
           </execution>
         </executions>
       </plugin>
@@ -261,7 +264,7 @@ See the Mulan PSL v2 for more details.
             <id>spotless-check</id>
             <phase>validate</phase>
             <goals>
-              <goal>apply</goal>
+              <goal>check</goal>
             </goals>
           </execution>
         </executions>