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

chengpan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new 7887a47  [KYUUBI #1996] Clean up log4j2 conf and enhance surefire 
plugin conf
7887a47 is described below

commit 7887a47e02be7b96ec43342c61fa7e7c960b841d
Author: Cheng Pan <[email protected]>
AuthorDate: Thu Mar 3 12:58:38 2022 +0800

    [KYUUBI #1996] Clean up log4j2 conf and enhance surefire plugin conf
    
    ### _Why are the changes needed?_
    
    [THRIFT-4805](https://issues.apache.org/jira/browse/THRIFT-4805) has been 
fixed in thrift 0.13.0, we can remove the workaround of log4j2 configurations 
because we use thrift 0.16.0 now, the change also removes the following 
warnings.
    
    ```
    2022-03-01 23:18:16,530 main ERROR Filters contains invalid attributes 
"onMatch", "onMismatch"
    2022-03-01 23:18:16,543 main ERROR Filters contains invalid attributes 
"onMatch", "onMismatch"
    ```
    
    `maven-surefire-plugin` is only used in `kyuubi-hive-jdbc`, we should 
disable it in other modules.
    
    ```
    [INFO] --- maven-surefire-plugin:2.22.0:test (default-test)  
kyuubi-ctl_2.12 ---
    [INFO]
    [INFO] -------------------------------------------------------
    [INFO]  T E S T S
    [INFO] -------------------------------------------------------
    2022-03-01 23:18:16,530 main ERROR Filters contains invalid attributes 
"onMatch", "onMismatch"
    2022-03-01 23:18:16,543 main ERROR Filters contains invalid attributes 
"onMatch", "onMismatch"
    [INFO] Running org.apache.kyuubi.ctl.ServiceControlCliSuite
    [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 
0.031 s - in org.apache.kyuubi.ctl.ServiceControlCliSuite
    [INFO] Running org.apache.kyuubi.ctl.ServiceControlCliArgumentsSuite
    [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 s 
- in org.apache.kyuubi.ctl.ServiceControlCliArgumentsSuite
    [INFO]
    [INFO] Results:
    [INFO]
    [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0
    ```
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run 
test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests)
 locally before make a pull request
    
    Closes #1996 from pan3793/log.
    
    Closes #1996
    
    291441c0 [Cheng Pan] nit
    f2c94246 [Cheng Pan] Tune surefire plugin in conf
    e1ee6402 [Cheng Pan] Clean up log4j2 conf
    
    Authored-by: Cheng Pan <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 conf/log4j2.properties.template                    | 13 ++---------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 docs/deployment/settings.md                        | 13 ++---------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/main/resources/log4j-defaults.properties   |  9 --------
 .../src/main/resources/log4j2-defaults.properties  | 13 ++---------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 kyuubi-hive-jdbc/pom.xml                           |  3 +++
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 .../src/test/resources/log4j2-test.properties      | 25 ++++------------------
 pom.xml                                            |  7 +-----
 20 files changed, 66 insertions(+), 342 deletions(-)

diff --git a/conf/log4j2.properties.template b/conf/log4j2.properties.template
index 102e0d3..f57c634 100644
--- a/conf/log4j2.properties.template
+++ b/conf/log4j2.properties.template
@@ -25,17 +25,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = info
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = info
 
 # Set the default kyuubi-ctl log level to WARN. When running the kyuubi-ctl, 
the
 # log level for this class is used to overwrite the root logger's log level.
diff --git 
a/dev/kyuubi-extension-spark-3-1/src/test/resources/log4j2-test.properties 
b/dev/kyuubi-extension-spark-3-1/src/test/resources/log4j2-test.properties
index fef13bd..b85b18e 100644
--- a/dev/kyuubi-extension-spark-3-1/src/test/resources/log4j2-test.properties
+++ b/dev/kyuubi-extension-spark-3-1/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git 
a/dev/kyuubi-extension-spark-3-2/src/test/resources/log4j2-test.properties 
b/dev/kyuubi-extension-spark-3-2/src/test/resources/log4j2-test.properties
index fef13bd..b85b18e 100644
--- a/dev/kyuubi-extension-spark-3-2/src/test/resources/log4j2-test.properties
+++ b/dev/kyuubi-extension-spark-3-2/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git 
a/dev/kyuubi-extension-spark-common/src/test/resources/log4j2-test.properties 
b/dev/kyuubi-extension-spark-common/src/test/resources/log4j2-test.properties
index fef13bd..b85b18e 100644
--- 
a/dev/kyuubi-extension-spark-common/src/test/resources/log4j2-test.properties
+++ 
b/dev/kyuubi-extension-spark-common/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git a/docs/deployment/settings.md b/docs/deployment/settings.md
index 8516898..fc07eb8 100644
--- a/docs/deployment/settings.md
+++ b/docs/deployment/settings.md
@@ -407,17 +407,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = info
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = info
 
 # Set the default kyuubi-ctl log level to WARN. When running the kyuubi-ctl, 
the
 # log level for this class is used to overwrite the root logger's log level.
diff --git 
a/externals/kyuubi-flink-sql-engine/src/test/resources/log4j2-test.properties 
b/externals/kyuubi-flink-sql-engine/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- 
a/externals/kyuubi-flink-sql-engine/src/test/resources/log4j2-test.properties
+++ 
b/externals/kyuubi-flink-sql-engine/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git 
a/externals/kyuubi-spark-sql-engine/src/test/resources/log4j2-test.properties 
b/externals/kyuubi-spark-sql-engine/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- 
a/externals/kyuubi-spark-sql-engine/src/test/resources/log4j2-test.properties
+++ 
b/externals/kyuubi-spark-sql-engine/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git 
a/externals/kyuubi-trino-engine/src/test/resources/log4j2-test.properties 
b/externals/kyuubi-trino-engine/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- a/externals/kyuubi-trino-engine/src/test/resources/log4j2-test.properties
+++ b/externals/kyuubi-trino-engine/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git 
a/integration-tests/kyuubi-flink-it/src/test/resources/log4j2-test.properties 
b/integration-tests/kyuubi-flink-it/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- 
a/integration-tests/kyuubi-flink-it/src/test/resources/log4j2-test.properties
+++ 
b/integration-tests/kyuubi-flink-it/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git 
a/integration-tests/kyuubi-kubernetes-deployment-it/src/test/resources/log4j2-test.properties
 
b/integration-tests/kyuubi-kubernetes-deployment-it/src/test/resources/log4j2-test.properties
index 73a9aa0..116202f 100644
--- 
a/integration-tests/kyuubi-kubernetes-deployment-it/src/test/resources/log4j2-test.properties
+++ 
b/integration-tests/kyuubi-kubernetes-deployment-it/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to INFO
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = info
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = info
diff --git a/kyuubi-common/src/main/resources/log4j-defaults.properties 
b/kyuubi-common/src/main/resources/log4j-defaults.properties
index 5354a8e..2d091a9 100644
--- a/kyuubi-common/src/main/resources/log4j-defaults.properties
+++ b/kyuubi-common/src/main/resources/log4j-defaults.properties
@@ -21,12 +21,3 @@ log4j.appender.console=org.apache.log4j.ConsoleAppender
 log4j.appender.console.target=System.err
 log4j.appender.console.layout=org.apache.log4j.PatternLayout
 log4j.appender.console.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss.SSS} %p 
%c{2}: %m%n
-
-# SPARK-34128: Suppress undesirable TTransportException warnings involved in 
THRIFT-4805
-log4j.appender.CA.filter.1=org.apache.log4j.varia.StringMatchFilter
-log4j.appender.CA.filter.1.StringToMatch=Thrift error occurred during 
processing of message
-log4j.appender.CA.filter.1.AcceptOnMatch=false
-
-log4j.appender.FA.filter.1=org.apache.log4j.varia.StringMatchFilter
-log4j.appender.FA.filter.1.StringToMatch=Thrift error occurred during 
processing of message
-log4j.appender.FA.filter.1.AcceptOnMatch=false
diff --git a/kyuubi-common/src/main/resources/log4j2-defaults.properties 
b/kyuubi-common/src/main/resources/log4j2-defaults.properties
index 9170c17..508d615 100644
--- a/kyuubi-common/src/main/resources/log4j2-defaults.properties
+++ b/kyuubi-common/src/main/resources/log4j2-defaults.properties
@@ -25,14 +25,5 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = info
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = info
diff --git a/kyuubi-common/src/test/resources/log4j2-test.properties 
b/kyuubi-common/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- a/kyuubi-common/src/test/resources/log4j2-test.properties
+++ b/kyuubi-common/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git a/kyuubi-ctl/src/test/resources/log4j2-test.properties 
b/kyuubi-ctl/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- a/kyuubi-ctl/src/test/resources/log4j2-test.properties
+++ b/kyuubi-ctl/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git a/kyuubi-ha/src/test/resources/log4j2-test.properties 
b/kyuubi-ha/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- a/kyuubi-ha/src/test/resources/log4j2-test.properties
+++ b/kyuubi-ha/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git a/kyuubi-hive-jdbc/pom.xml b/kyuubi-hive-jdbc/pom.xml
index 7413e5a..0fdd98a 100644
--- a/kyuubi-hive-jdbc/pom.xml
+++ b/kyuubi-hive-jdbc/pom.xml
@@ -241,6 +241,9 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <skipTests>false</skipTests>
+                </configuration>
             </plugin>
 
             <plugin>
diff --git a/kyuubi-metrics/src/test/resources/log4j2-test.properties 
b/kyuubi-metrics/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- a/kyuubi-metrics/src/test/resources/log4j2-test.properties
+++ b/kyuubi-metrics/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git a/kyuubi-server/src/test/resources/log4j2-test.properties 
b/kyuubi-server/src/test/resources/log4j2-test.properties
index 73a9aa0..116202f 100644
--- a/kyuubi-server/src/test/resources/log4j2-test.properties
+++ b/kyuubi-server/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to INFO
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = info
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = info
diff --git a/kyuubi-zookeeper/src/test/resources/log4j2-test.properties 
b/kyuubi-zookeeper/src/test/resources/log4j2-test.properties
index 3975a52..0c18971 100644
--- a/kyuubi-zookeeper/src/test/resources/log4j2-test.properties
+++ b/kyuubi-zookeeper/src/test/resources/log4j2-test.properties
@@ -26,17 +26,8 @@ appender.console.name = STDOUT
 appender.console.target = SYSTEM_OUT
 appender.console.layout.type = PatternLayout
 appender.console.layout.pattern = %d{HH:mm:ss.SSS} %p %c: %m%n
-
-appender.console.filter.1.type = Filters
-
-appender.console.filter.1.a.type = ThresholdFilter
-appender.console.filter.1.a.level = fatal
-
-# SPARK-34128: Suppress undesirable TTransportException warnings, due to 
THRIFT-4805
-appender.console.filter.1.b.type = RegexFilter
-appender.console.filter.1.b.regex = .*Thrift error occurred during processing 
of message.*
-appender.console.filter.1.b.onMatch = deny
-appender.console.filter.1.b.onMismatch = neutral
+appender.console.filter.a.type = ThresholdFilter
+appender.console.filter.a.level = fatal
 
 # File Appender
 appender.file.type = File
@@ -44,14 +35,6 @@ appender.file.name = File
 appender.file.fileName = target/unit-tests.log
 appender.file.layout.type = PatternLayout
 appender.file.layout.pattern = %d{HH:mm:ss.SSS} %t %p %c{1}: %m%n
-
-appender.file.filter.1.type = Filters
-
-appender.file.filter.1.a.type = RegexFilter
-appender.file.filter.1.a.regex = .*Thrift error occurred during processing of 
message.*
-appender.file.filter.1.a.onMatch = deny
-appender.file.filter.1.a.onMismatch = neutral
-
 # Set the logger level of File Appender to DEBUG
-appender.file.filter.1.b.type = ThresholdFilter
-appender.file.filter.1.b.level = debug
+appender.file.filter.a.type = ThresholdFilter
+appender.file.filter.a.level = debug
diff --git a/pom.xml b/pom.xml
index fc72450..0c405d0 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1469,12 +1469,7 @@
                     <artifactId>maven-surefire-plugin</artifactId>
                     <version>${maven.plugin.surefire.version}</version>
                     <configuration>
-                        <includes>
-                            <include>**/Test*.java</include>
-                            <include>**/*Test.java</include>
-                            <include>**/*TestCase.java</include>
-                            <include>**/*Suite.java</include>
-                        </includes>
+                        <skipTests>true</skipTests>
                         <failIfNoTests>false</failIfNoTests>
                     </configuration>
                 </plugin>

Reply via email to