You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2022/09/14 09:44:25 UTC

[flink-connector-elasticsearch] 01/03: [hotfix] Wipe checkstyle suppressions

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

chesnay pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/flink-connector-elasticsearch.git

commit f2be867c51739d264454c2ed1a4ad6cd1048f0b1
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Wed Sep 14 11:27:33 2022 +0200

    [hotfix] Wipe checkstyle suppressions
---
 tools/maven/suppressions.xml | 59 --------------------------------------------
 1 file changed, 59 deletions(-)

diff --git a/tools/maven/suppressions.xml b/tools/maven/suppressions.xml
index 129a0c7..4f80ec2 100644
--- a/tools/maven/suppressions.xml
+++ b/tools/maven/suppressions.xml
@@ -23,63 +23,4 @@ under the License.
 		"http://www.puppycrawl.com/dtds/suppressions_1_1.dtd">
 
 <suppressions>
-		<!-- 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=".*"/>
-		<!-- target directory is not relevant for checkstyle -->
-		<suppress
-			files="[\\/]target[\\/]"
-			checks=".*"/>
-		<suppress
-			files="org[\\/]apache[\\/]flink[\\/]formats[\\/]avro[\\/]glue[\\/]schema[\\/]registry[\\/]User.java"
-			checks=".*"/>
 </suppressions>