build: format pom and code

.github/workflows/build.yml

@@ -1,3 +1,17 @@
+# Copyright 2023 Greptime Team
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 name: build
 
 on:
@@ -7,57 +21,59 @@ on:
     branches: [ main ]
 
 jobs:
-  check_format:
+  check:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: Set up JDK 1.8
-        uses: actions/setup-java@v1
+      - uses: actions/checkout@v4
+      - uses: korandoru/hawkeye@v4
+      - uses: actions/setup-java@v4
         with:
-          java-version: 1.8
+          java-version: '21'
+          distribution: 'zulu'
       - name: Check format
-        run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
-          && sh ./tools/check_format.sh
+        run: mvn spotless:check
+      - name: Check Sort POM
+        run: mvn sortpom:verify
 
   test_ingester_common:
-    needs: check_format
+    needs: check
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: Set up JDK 1.8
-        uses: actions/setup-java@v1
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          java-version: 1.8
+          java-version: '8'
+          distribution: 'zulu'
       - name: Maven Test
         run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
           && (mvn --projects ingester-common test
             || mvn --projects ingester-common test
             || mvn --projects ingester-common test)
 
   test_ingester_grpc:
-    needs: check_format
+    needs: check
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: Set up JDK 1.8
-        uses: actions/setup-java@v1
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          java-version: 1.8
+          java-version: '8'
+          distribution: 'zulu'
       - name: Maven Test
         run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
           && (mvn --projects ingester-grpc test
             || mvn --projects ingester-grpc test
             || mvn --projects ingester-grpc test)
 
   test_ingester_protocol:
-    needs: check_format
+    needs: check
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: Set up JDK 1.8
-        uses: actions/setup-java@v1
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          java-version: 1.8
+          java-version: '8'
+          distribution: 'zulu'
       - name: Maven Test
         run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
           && (mvn --projects ingester-protocol test

.github/workflows/mvn_publish.yml

@@ -1,3 +1,17 @@
+# Copyright 2023 Greptime Team
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 name: Maven Publish
 
 on:
@@ -8,13 +22,11 @@ jobs:
   publish:
     runs-on: ubuntu-latest
     steps:
-      - name: Checkout Git Repo
-        uses: actions/checkout@v2
-
-      - name: Set up JDK 1.8
-        uses: actions/setup-java@v1
+      - uses: actions/checkout@v4
+      - uses: actions/setup-java@v4
         with:
-          java-version: 1.8
+          java-version: '8'
+          distribution: 'zulu'
           server-id: ossrh
           server-username: MAVEN_USERNAME
           server-password: MAVEN_PASSWORD

.gitignore

@@ -27,6 +27,7 @@ target
 
 # dev tools
 .idea
+*.iml
 
 *.bak
 .vscode/*

ingester-all/pom.xml

@@ -1,20 +1,31 @@
 <?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+    Copyright 2023 Greptime Team
+
+    Licensed under the Apache License, Version 2.0 (the "License");
+    you may not use this file except in compliance with the License.
+    You may obtain a copy of the License at
+
+        http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing, software
+    distributed under the License is distributed on an "AS IS" BASIS,
+    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+    See the License for the specific language governing permissions and
+    limitations under the License.
+
+-->
 <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 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
     <parent>
-        <artifactId>greptimedb-ingester</artifactId>
         <groupId>io.greptime</groupId>
+        <artifactId>greptimedb-ingester</artifactId>
         <version>0.7.3</version>
     </parent>
 
     <artifactId>ingester-all</artifactId>
 
-    <properties>
-        <maven.compiler.source>8</maven.compiler.source>
-        <maven.compiler.target>8</maven.compiler.target>
-        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-    </properties>
-
     <dependencies>
         <dependency>
             <groupId>${project.groupId}</groupId>

ingester-common/pom.xml

@@ -1,19 +1,33 @@
 <?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+    Copyright 2023 Greptime Team
+
+    Licensed under the Apache License, Version 2.0 (the "License");
+    you may not use this file except in compliance with the License.
+    You may obtain a copy of the License at
+
+        http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing, software
+    distributed under the License is distributed on an "AS IS" BASIS,
+    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+    See the License for the specific language governing permissions and
+    limitations under the License.
+
+-->
 <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 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
     <parent>
-        <artifactId>greptimedb-ingester</artifactId>
         <groupId>io.greptime</groupId>
+        <artifactId>greptimedb-ingester</artifactId>
         <version>0.7.3</version>
     </parent>
 
     <artifactId>ingester-common</artifactId>
 
     <properties>
-        <maven.compiler.source>8</maven.compiler.source>
-        <maven.compiler.target>8</maven.compiler.target>
         <metrics.version>4.2.11</metrics.version>
-        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     </properties>
 
     <dependencies>
@@ -35,8 +49,8 @@
             <version>${metrics.version}</version>
             <exclusions>
                 <exclusion>
-                    <artifactId>slf4j-api</artifactId>
                     <groupId>org.slf4j</groupId>
+                    <artifactId>slf4j-api</artifactId>
                 </exclusion>
             </exclusions>
         </dependency>

ingester-common/src/main/java/com/google/protobuf/ByteStringHelper.java

@@ -13,12 +13,11 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package com.google.protobuf;
 
 /**
  * A {@code ByteString} helper, avoid some memory copying to improve performance.
- *
- * @author jiachun.fjc
  */
 public class ByteStringHelper {
 

ingester-common/src/main/java/io/greptime/common/Copiable.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 /**

ingester-common/src/main/java/io/greptime/common/Display.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 import java.io.PrintWriter;
@@ -21,8 +22,6 @@
  * Components that implement this interface need to be able to display
  * their own state and output state information via the {@code display}
  * method.
- *
- * @author jiachun.fjc
  */
 public interface Display {
 

ingester-common/src/main/java/io/greptime/common/Endpoint.java

@@ -13,15 +13,14 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 import io.greptime.common.util.Strings;
 import java.io.Serializable;
 
 /**
  * An IP address with port.
- *
- * @author jiachun.fjc
  */
 public class Endpoint implements Serializable {
 

ingester-common/src/main/java/io/greptime/common/Into.java

@@ -13,12 +13,11 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 /**
  * A value-to-value conversion.
- *
- * @author jiachun.fjc
  */
 public interface Into<Output> {
     Output into();

ingester-common/src/main/java/io/greptime/common/Keys.java

@@ -13,12 +13,11 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 /**
  * Constant string keys.
- *
- * @author jiachun.fjc
  */
 public final class Keys {
     public static final String DB_NAME = "GreptimeDB";

ingester-common/src/main/java/io/greptime/common/Lifecycle.java

@@ -13,12 +13,11 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 /**
  * Service life cycle mark interface.
- *
- * @author jiachun.fjc
  */
 public interface Lifecycle<T> {
 

ingester-common/src/main/java/io/greptime/common/SPI.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common;
 
 import java.lang.annotation.Documented;
@@ -23,8 +24,6 @@
 
 /**
  * Service provide interface annotation.
- *
- * @author jiachun.fjc
  */
 @Documented
 @Retention(RetentionPolicy.RUNTIME)

ingester-common/src/main/java/io/greptime/common/signal/FileOutputHelper.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common.signal;
 
 import io.greptime.common.Keys;
@@ -26,8 +27,6 @@
 
 /**
  * File output helper.
- *
- * @author jiachun.fjc
  */
 public final class FileOutputHelper {
 

ingester-common/src/main/java/io/greptime/common/signal/FileSignal.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package io.greptime.common.signal;
 
 import java.util.Optional;
@@ -23,12 +24,10 @@
  * Adopt the method of creating files with specified names to interact
  * with the Client process and implement signal transmission, achieve the
  * purpose of controlling the process to output specified content through this.
- *
- * @author jiachun.fjc
  */
 public enum FileSignal {
-    RwLogging("rw_logging.sig", "How to open or close read/write log(The second execution means close)"), //
-    RpcLimit("rpc_limit.sig", "How to open or close rpc limiter(The second execution means close)"), //
+    RwLogging("rw_logging.sig", "How to open or close read/write log(The second execution means close)"),
+    RpcLimit("rpc_limit.sig", "How to open or close rpc limiter(The second execution means close)"),
     ;
 
     private final String filename;