fix checkstyle

Minor fix file check

fix (#2327)

Author: lionelcao

dev-support/checkstyle-apache.xml

@@ -28,10 +28,17 @@
     </module>
 
     <module name="Header">
-        <property name="headerFile" value="dev-support/checkstyle-apache-header.txt"/>
-        <property name="fileExtensions" value="java"/>
+        <property name="headerFile" value="dev-support/checkstyle-java-header.txt"/>
+        <property name="fileExtensions" value="java, scala"/>
+    </module>
+    <module name="Header">
+        <property name="headerFile" value="dev-support/checkstyle-prop-header.txt"/>
+        <property name="fileExtensions" value="properties"/>
+    </module>
+    <module name="Header">
+        <property name="headerFile" value="dev-support/checkstyle-xml-header.txt"/>
+        <property name="fileExtensions" value="xml"/>
     </module>
-
     <module name="FileTabCharacter"/>
     <module name="SuppressWarningsFilter"/>
     <module name="NewlineAtEndOfFile"/>

dev-support/checkstyle-apache-header.txt renamed to dev-support/checkstyle-java-header.txt

@@ -0,0 +1,16 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You 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.
+#

dev-support/checkstyle-prop-header.txt

@@ -0,0 +1,16 @@
+<!--
+  Licensed to the Apache Software Foundation (ASF) under one or more
+  contributor license agreements.  See the NOTICE file distributed with
+  this work for additional information regarding copyright ownership.
+  The ASF licenses this file to You 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.
+-->

dev-support/checkstyle-xml-header.txt

@@ -14,7 +14,8 @@
   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/maven-v4_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 http://maven.apache.org/maven-v4_0_0.xsd">
 
     <parent>
         <!-- https://infra.apache.org/publishing-maven-artifacts.html -->
@@ -3390,6 +3391,9 @@
                                 <configLocation>dev-support/checkstyle-apache.xml</configLocation>
                                 <suppressionsLocation>dev-support/checkstyle-apache-suppressions.xml
                                 </suppressionsLocation>
+                                <includes>
+                                    src/main/java/**/*.java,src/main/scala/**/*.scala,src/main/scala/**/*.java,src/test/java/**/*.java,src/test/scala/**/*.scala,src/test/scala/**/*.java
+                                </includes>
                                 <includeTestSourceDirectory>true</includeTestSourceDirectory>
                                 <consoleOutput>true</consoleOutput>
                                 <failsOnError>true</failsOnError>
@@ -3972,23 +3976,6 @@
                                 <exclude>**/*.bmp</exclude>
 
                                 <exclude>src/main/codegen/javacc/org/apache/kylin/query/util/CommentParser.jj</exclude>
-
-                                <!-- protobuf generated -->
-                                <exclude>
-                                    src/main/java/org/apache/kylin/storage/hbase/ii/coprocessor/endpoint/generated/IIProtos.java
-                                </exclude>
-                                <exclude>
-                                    src/main/java/org/apache/kylin/storage/hbase/cube/v1/filter/generated/FilterProtosExt.java
-                                </exclude>
-                                <exclude>
-                                    src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/generated/CubeVisitProtos.java
-                                </exclude>
-                                <exclude>
-                                    storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/coprocessor/endpoint/generated/CubeVisitProtos.java
-                                </exclude>
-                                <exclude>
-                                    examples/test_case_data/sample_local/UUID
-                                </exclude>
                             </excludes>
                         </configuration>
                         <executions>

pom.xml

@@ -14,6 +14,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
+
 kylin.metadata.url=${KYLIN_METADATA_URL}
 kylin.metadata.audit-log.max-size=3000000
 kylin.metadata.ops-cron=0 0 0 * * *

src/common-booter/src/main/resources/config/init_min.properties

@@ -228,8 +228,8 @@ public void batchInsert(List<AuditLog> auditLogs) {
                     try {
                         val bs = Objects.isNull(x.getByteSource()) ? null : x.getByteSource().read();
                         return new Object[] { x.getResPath(), CompressionUtils.compress(bs), x.getTimestamp(),
-                                x.getMvcc(), x.getUnitId(), x.getModelUuid(), x.getOperator(), x.getInstance(), x.getProject(),
-                                x.isDiffFlag() };
+                                x.getMvcc(), x.getUnitId(), x.getModelUuid(), x.getOperator(), x.getInstance(),
+                                x.getProject(), x.isDiffFlag() };
                     } catch (IOException e) {
                         return null;
                     }

src/core-common/src/main/java/org/apache/kylin/common/persistence/metadata/JdbcAuditLogStore.java

@@ -17,8 +17,6 @@
  */
 package org.apache.kylin.common.persistence.metadata;
 
-import java.util.function.Predicate;
-
 import org.apache.kylin.common.KylinConfig;
 
 import lombok.extern.slf4j.Slf4j;

src/core-common/src/main/java/org/apache/kylin/common/persistence/metadata/JdbcPartialAuditLogStore.java

@@ -43,22 +43,18 @@
 import org.apache.kylin.common.persistence.RawResourceFilter;
 import org.apache.kylin.common.persistence.metadata.mapper.BasicMapper;
 import org.apache.kylin.common.persistence.metadata.mapper.BasicSqlTable;
+import org.apache.kylin.guava30.shaded.common.base.Preconditions;
 import org.mybatis.dynamic.sql.BasicColumn;
 import org.mybatis.dynamic.sql.BindableColumn;
 import org.mybatis.dynamic.sql.SqlBuilder;
 import org.mybatis.dynamic.sql.VisitableCondition;
 import org.mybatis.dynamic.sql.select.SelectDSLCompleter;
 import org.mybatis.spring.transaction.SpringManagedTransactionFactory;
 
-import com.google.common.base.Preconditions;
-
 import lombok.val;
 import lombok.var;
 import lombok.extern.slf4j.Slf4j;
 
-/**
- * 15/11/2023 hellozepp([email protected])
- */
 @Slf4j
 public class MetadataMapperFactory {
 

src/core-common/src/main/java/org/apache/kylin/common/persistence/metadata/MetadataMapperFactory.java

@@ -95,8 +95,10 @@ public void dump(ResourceStore store, Collection<String> resources) throws Excep
     }
 
     /**
-     * upload local files to snapshot, will not change resource store synchronized, perhaps should call ResourceStore.clearCache() manually
-     * @param folder local directory contains snapshot, ${folder}/events contains event store, ${folder}/kylin.properties etc.
+     * upload local files to snapshot, will not change resource store synchronized,
+     * perhaps should call ResourceStore.clearCache() manually
+     * @param folder local directory contains snapshot, ${folder}/events contains event store,
+     * ${folder}/kylin.properties etc.
      */
     public void uploadFromFile(File folder) {
         foreachFile(folder, res -> {