remove unused/old suppressions. reduce scope on some

confluentinc · Mar 13, 2018 · 31e6970 · 31e6970
1 parent 8bc3559
commit 31e6970
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 13 deletions.
diff --git a/checkstyle/suppressions.xml b/checkstyle/suppressions.xml
@@ -7,16 +7,11 @@
 <suppressions>
 
     <!-- Checkstyle 6.19 doesn't like properly-formatted annotations -->
-    <suppress files="(KsqlEntity|Cli|Ksql|AbstractCliCommands|CommandStore).java" checks="IndentationCheck"/>
-
-    <!-- can't work out how to resolve the issue with the do {..} while -->
-    <suppress files="CommandStore.java" checks="RightCurly"/>
+    <suppress files="KsqlEntity.java" checks="IndentationCheck"/>
 
     <!-- Suppress some checks on all files -->
-    <suppress files="." checks="ExplicitInitializationCheck"/>
-    <suppress files="." checks="FinalClassCheck"/>
     <suppress files="." checks="AbbreviationAsWordInName"/>
-    <suppress files="." checks="AvoidEscapedUnicodeCharacters"/>
+    <suppress files="JsonPathTokenizer.java" checks="AvoidEscapedUnicodeCharacters"/>
 
     <!-- Semantics may not be well checked by rules -->
     <suppress files="." checks="ParameterNumber"/>

diff --git a/ksql-cli/src/main/java/io/confluent/ksql/Ksql.java b/ksql-cli/src/main/java/io/confluent/ksql/Ksql.java
@@ -54,8 +54,9 @@ public static void main(String[] args) throws IOException {
     try (final Cli cli = new Cli(options.getStreamedQueryRowLimit(),
         options.getStreamedQueryTimeoutMs(),
         restClient,
-        new JLineTerminal(options.getOutputFormat(), restClient))) {
-        cli.runInteractively();
+        new JLineTerminal(options.getOutputFormat(), restClient))
+    ) {
+      cli.runInteractively();
     }
   }
 

diff --git a/ksql-cli/src/main/java/io/confluent/ksql/cli/Cli.java b/ksql-cli/src/main/java/io/confluent/ksql/cli/Cli.java
@@ -289,8 +289,7 @@ private void handleStatements(String line)
       throws IOException, InterruptedException, ExecutionException {
     StringBuilder consecutiveStatements = new StringBuilder();
     for (SqlBaseParser.SingleStatementContext statementContext :
-        new KsqlParser().getStatements(line)
-        ) {
+        new KsqlParser().getStatements(line)) {
       String statementText = KsqlEngine.getStatementString(statementContext);
       if (statementContext.statement() instanceof SqlBaseParser.QuerystatementContext
           || statementContext.statement() instanceof SqlBaseParser.PrintTopicContext) {
@@ -402,8 +401,8 @@ private void printKsqlResponse(RestResponse<KsqlEntityList> response) throws IOE
         } else if (entity instanceof CommandStatusEntity
             && (
             ((CommandStatusEntity) entity).getCommandStatus().getStatus()
-                == CommandStatus.Status.ERROR
-        )) {
+                == CommandStatus.Status.ERROR)
+        ) {
           String fullMessage = ((CommandStatusEntity) entity).getCommandStatus().getMessage();
           terminal.printError(fullMessage.split("\n")[0], fullMessage);
           noErrorFromServer = false;