Various checkstyle compaints

linkedin · Feb 7, 2025 · 58931aa · 58931aa
1 parent 33f0a96
commit 58931aa
Show file tree

Hide file tree

Showing 8 changed files with 10 additions and 17 deletions.
diff --git a/deploy/samples/kafkadb.yaml b/deploy/samples/kafkadb.yaml
@@ -37,6 +37,8 @@ spec:
     properties.bootstrap.servers = one-kafka-bootstrap.kafka.svc.cluster.local:9094
     value.format = json
     scan.startup.mode = earliest-offset
+    key.fields = KEY
+    value.fields-include = EXCEPT_KEY
 
 ---
 

diff --git a/deploy/samples/venicedb.yaml b/deploy/samples/venicedb.yaml
@@ -23,4 +23,4 @@ spec:
     key.fields-prefix = {{keyPrefix:}}
     key.fields = {{keys:KEY}}
     key.type = {{keyType:PRIMITIVE}}
-    value.fields-include: EXCEPT_KEY
+    value.fields-include = EXCEPT_KEY
diff --git a/hoptimator-jdbc/src/main/java/com/linkedin/hoptimator/jdbc/BuiltinCatalogProvider.java b/hoptimator-jdbc/src/main/java/com/linkedin/hoptimator/jdbc/BuiltinCatalogProvider.java
@@ -2,7 +2,6 @@
 
 import java.util.Collection;
 import java.util.Collections;
-import java.util.Properties;
 
 import com.linkedin.hoptimator.Catalog;
 import com.linkedin.hoptimator.CatalogProvider;

diff --git a/hoptimator-jdbc/src/main/java/com/linkedin/hoptimator/jdbc/HoptimatorConnection.java b/hoptimator-jdbc/src/main/java/com/linkedin/hoptimator/jdbc/HoptimatorConnection.java
@@ -1,8 +1,7 @@
 package com.linkedin.hoptimator.jdbc;
 
-import java.sql.Connection;
-import java.sql.Statement;
 import java.sql.SQLException;
+import java.sql.Statement;
 import java.util.Properties;
 
 import org.apache.calcite.jdbc.CalciteConnection;
@@ -24,15 +23,15 @@ public HoptimatorConnection(CalciteConnection connection, Properties connectionP
 
   @Override
   public Statement createStatement() throws SQLException {
-    return connection.createStatement(); 
+    return connection.createStatement();
   }
 
   public Properties connectionProperties() {
     return connectionProperties;
   }
 
   public CalcitePrepare.Context createPrepareContext() {
-    return connection.createPrepareContext(); 
+    return connection.createPrepareContext();
   }
 
   public CalciteConnection calciteConnection() {

diff --git a/hoptimator-jdbc/src/main/java/com/linkedin/hoptimator/jdbc/HoptimatorDdlExecutor.java b/hoptimator-jdbc/src/main/java/com/linkedin/hoptimator/jdbc/HoptimatorDdlExecutor.java
@@ -173,7 +173,7 @@ public void execute(SqlCreateMaterializedView create, CalcitePrepare.Context con
       MaterializedViewTable materializedViewTable = new MaterializedViewTable(viewTableMacro);
       RelDataType viewRowType = materializedViewTable.getRowType(typeFactory);
 
-      // Suport "partial views", i.e. CREATE VIEW FOO$BAR, where the view name
+      // Support "partial views", i.e. CREATE VIEW FOO$BAR, where the view name
       // is "foo-bar" and the sink is just FOO.
       String sinkName = viewName.split("\\$", 2)[0];
       List<String> sinkPath = new ArrayList<>();

diff --git a/hoptimator-k8s/src/main/java/com/linkedin/hoptimator/k8s/K8sContext.java b/hoptimator-k8s/src/main/java/com/linkedin/hoptimator/k8s/K8sContext.java
@@ -1,19 +1,14 @@
 package com.linkedin.hoptimator.k8s;
 
 import java.io.File;
-import java.io.InputStream;
 import java.io.IOException;
+import java.io.InputStream;
 import java.io.Reader;
 import java.nio.file.Files;
-import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.time.Duration;
-import java.util.Optional;
 import java.util.Properties;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import io.kubernetes.client.apimachinery.GroupVersion;
 import io.kubernetes.client.common.KubernetesListObject;
 import io.kubernetes.client.common.KubernetesObject;
@@ -86,12 +81,12 @@ public K8sContext(Properties connectionProperties) {
         throw new RuntimeException(e);
       }
     }
- 
+
     if (server != null) {
       info += " Accessing " + server + ".";
       this.apiClient.setBasePath(server);
     }
- 
+
     if (truststore != null) {
       try {
         InputStream in = Files.newInputStream(Paths.get(truststore));

diff --git a/...imator-operator/src/main/java/com/linkedin/hoptimator/operator/HoptimatorOperatorApp.java b/...imator-operator/src/main/java/com/linkedin/hoptimator/operator/HoptimatorOperatorApp.java
@@ -21,7 +21,6 @@
 import io.kubernetes.client.extended.controller.ControllerManager;
 import io.kubernetes.client.informer.SharedInformerFactory;
 import io.kubernetes.client.openapi.ApiClient;
-import io.kubernetes.client.util.Config;
 
 import com.linkedin.hoptimator.catalog.Resource;
 import com.linkedin.hoptimator.k8s.K8sApiEndpoints;

diff --git a/hoptimator-util/src/main/java/com/linkedin/hoptimator/util/planner/ScriptImplementor.java b/hoptimator-util/src/main/java/com/linkedin/hoptimator/util/planner/ScriptImplementor.java
@@ -5,7 +5,6 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
-import java.util.Objects;
 import java.util.Optional;
 import java.util.Set;
 import java.util.function.Function;