[build][hotfix] Remove useless suppressions. (#2817)

apache · Dec 5, 2023 · 18fb0d8 · 18fb0d8
1 parent 63b39ba
commit 18fb0d8
Showing 1 changed file with 6 additions and 64 deletions.
diff --git 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>