From 18fb0d8b7f5eb310909339dad65241b52fca6245 Mon Sep 17 00:00:00 2001
From: Leonard Xu <leonard@apache.org>
Date: Tue, 5 Dec 2023 20:30:49 +0800
Subject: [PATCH] [build][hotfix] Remove useless suppressions. (#2817)

---
 tools/maven/suppressions.xml | 70 ++++--------------------------------
 1 file changed, 6 insertions(+), 64 deletions(-)

diff --git a/tools/maven/suppressions.xml b/tools/maven/suppressions.xml
index 1f9fe9a1c..c64a07d42 100644
--- a/tools/maven/suppressions.xml
+++ b/tools/maven/suppressions.xml
@@ -24,68 +24,10 @@ under the License.
 	<!-- Skip MethodNameCheck for DataTypes	-->
 	<suppress files="DataTypes.java" checks="MethodNameCheck"/>
 
-	<!-- Star import is used for all the expressions -->
-	<suppress files="BaseExpressions.java" checks="AvoidStarImport"/>
-
-	<!-- These use star import for all the generated Tuple classes -->
-	<suppress files="CsvReader.java" checks="AvoidStarImport"/>
-
-	<suppress files="NoticeFileChecker.java" checks="Regexp"/>
-	<suppress files="NoticeFileChecker.java" checks="IllegalImport"/>
-
-	<suppress files="JoinOperator.java" checks="FileLength"/>
-		<suppress files="WindowOperatorTest.java" checks="FileLength"/>
-		<suppress files="WindowOperatorContractTest.java" checks="FileLength"/>
-		<suppress files="NFAITCase.java" checks="FileLength"/>
-
-		<suppress files="org[\\/]apache[\\/]flink[\\/]formats[\\/]avro[\\/]generated[\\/].*.java" checks="[a-zA-Z0-9]*"/>
-		<suppress files="org[\\/]apache[\\/]flink[\\/]formats[\\/]parquet[\\/]generated[\\/].*.java" checks="[a-zA-Z0-9]*"/>
-		<!-- Sometimes we have to temporarily fix very long, different formatted Calcite files. -->
-		<suppress files="org[\\/]apache[\\/]calcite.*" checks="[a-zA-Z0-9]*"/>
-
-		<!-- Temporarily fix TM Metaspace memory leak caused by Apache Beam sdk harness. -->
-		<suppress files="org[\\/]apache[\\/]beam.*.java" checks="[a-zA-Z0-9]*"/>
-
-	    <!-- Have to use guava directly -->
-	    <suppress
-			files="OverConvertRule.java|CustomizedConvertRule.java"
-			checks="IllegalImport"/>
-		<!-- Cassandra connectors have to use guava directly -->
-		<suppress
-			files="AbstractCassandraTupleSink.java|CassandraInputFormat.java|CassandraOutputFormatBase.java|CassandraSinkBase.java|CassandraSinkBaseTest.java|CassandraPojoSink.java|CassandraRowSink.java|CassandraTupleWriteAheadSink.java|CassandraRowWriteAheadSink.java|CassandraPojoOutputFormat.java"
-			checks="IllegalImport"/>
-		<!-- Kinesis producer has to use guava directly -->
-		<suppress
-			files="FlinkKinesisProducer.java|FlinkKinesisProducerTest.java"
-			checks="IllegalImport"/>
-		<!-- Kinesis EFO consumer required to handle Netty ReadTimeoutException -->
-		<suppress 
-			files="FanOutRecordPublisherTest.java|FanOutShardSubscriber.java|FanOutShardSubscriberTest.java" 
-			checks="IllegalImport"/>
-	 	<!-- Classes copied from Hadoop -->
-		<suppress
-			files="org[\\/]apache[\\/]hadoop[\\/]conf[\\/]Configuration.java"
-			checks=".*"/>
-		<suppress
-			files="org[\\/]apache[\\/]hadoop[\\/]util[\\/]NativeCodeLoader.java"
-			checks=".*"/>
-		<suppress
-			files="org[\\/]apache[\\/]hadoop[\\/]util[\\/]VersionInfo.java"
-			checks=".*"/>
-		<!-- Classes copied from AWS -->
-		<suppress
-			files="com[\\/]amazonaws[\\/]services[\\/]s3[\\/]model[\\/]transform[\\/]XmlResponsesSaxParser.java"
-			checks=".*"/>
-
-		<!-- Classes copied from debezium -->
-		<suppress
-				files="io[\\/]debezium[\\/]connector[\\/]postgresql[\\/]"
-				checks=".*"/>
-		<suppress
-				files="io[\\/]debezium[\\/]"
-				checks=".*"/>
-		<!-- target directory is not relevant for checkstyle -->
-		<suppress
-			files="[\\/]target[\\/]"
-			checks=".*"/>
+	<!-- Classes copied from debezium -->
+	<suppress files="io[\\/]debezium[\\/]connector[\\/]postgresql[\\/]" checks=".*"/>
+	<suppress files="io[\\/]debezium[\\/]" checks=".*"/>
+	
+	<!-- target directory is not relevant for checkstyle -->
+	<suppress files="[\\/]target[\\/]" checks=".*"/>
 </suppressions>