Use static import for JsonCodec factory methods

Author: wendigo

core/trino-main/src/main/java/io/trino/operator/scalar/FailureFunction.java

@@ -23,9 +23,11 @@
 import io.trino.spi.function.SqlType;
 import io.trino.spi.type.StandardTypes;
 
+import static io.airlift.json.JsonCodec.jsonCodec;
+
 public final class FailureFunction
 {
-    private static final JsonCodec<FailureInfo> JSON_CODEC = JsonCodec.jsonCodec(FailureInfo.class);
+    private static final JsonCodec<FailureInfo> JSON_CODEC = jsonCodec(FailureInfo.class);
 
     private FailureFunction() {}
 

core/trino-main/src/main/java/io/trino/server/protocol/spooling/SpooledMetadataBlockSerde.java

@@ -27,6 +27,7 @@
 import java.util.Optional;
 
 import static com.google.common.base.Verify.verify;
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static io.airlift.json.JsonCodec.listJsonCodec;
 import static io.airlift.json.JsonCodec.mapJsonCodec;
 import static io.airlift.slice.Slices.utf8Slice;
@@ -44,7 +45,7 @@
 public class SpooledMetadataBlockSerde
 {
     private static final JsonCodec<Map<String, List<String>>> HEADERS_CODEC = mapJsonCodec(String.class, listJsonCodec(String.class));
-    private static final JsonCodec<DataAttributes> ATTRIBUTES_CODEC = JsonCodec.jsonCodec(DataAttributes.class);
+    private static final JsonCodec<DataAttributes> ATTRIBUTES_CODEC = jsonCodec(DataAttributes.class);
 
     private static final RowType SPOOLING_METADATA_TYPE = RowType.from(List.of(
             new RowType.Field(Optional.empty(), VARCHAR),

core/trino-main/src/main/java/io/trino/server/security/jwt/JwkDecoder.java

@@ -34,13 +34,14 @@
 import java.util.function.Function;
 
 import static com.google.common.collect.ImmutableMap.toImmutableMap;
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static java.util.Base64.getUrlDecoder;
 import static java.util.Objects.requireNonNull;
 
 public final class JwkDecoder
 {
     private static final Logger log = Logger.get(JwkDecoder.class);
-    private static final JsonCodec<Keys> KEYS_CODEC = JsonCodec.jsonCodec(Keys.class);
+    private static final JsonCodec<Keys> KEYS_CODEC = jsonCodec(Keys.class);
 
     private JwkDecoder() {}
 

core/trino-main/src/main/java/io/trino/sql/planner/planprinter/JsonRenderer.java

@@ -25,12 +25,13 @@
 import java.util.Optional;
 
 import static com.google.common.collect.ImmutableList.toImmutableList;
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static java.util.Objects.requireNonNull;
 
 public class JsonRenderer
         implements Renderer<String>
 {
-    private static final JsonCodec<JsonRenderedNode> CODEC = JsonCodec.jsonCodec(JsonRenderedNode.class);
+    private static final JsonCodec<JsonRenderedNode> CODEC = jsonCodec(JsonRenderedNode.class);
 
     @Override
     public String render(PlanRepresentation plan)

core/trino-main/src/test/java/io/trino/dispatcher/TestLocalDispatchQuery.java

@@ -18,7 +18,6 @@
 import com.google.common.util.concurrent.Futures;
 import com.google.common.util.concurrent.ListenableFuture;
 import io.airlift.configuration.secrets.SecretsResolver;
-import io.airlift.json.JsonCodec;
 import io.airlift.node.NodeInfo;
 import io.airlift.units.Duration;
 import io.trino.connector.CatalogHandle;
@@ -75,6 +74,7 @@
 
 import static com.google.common.util.concurrent.Futures.immediateVoidFuture;
 import static io.airlift.concurrent.Threads.daemonThreadsNamed;
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static io.airlift.tracing.Tracing.noopTracer;
 import static io.opentelemetry.api.OpenTelemetry.noop;
 import static io.trino.SessionTestUtils.TEST_SESSION;
@@ -127,10 +127,10 @@ public void testSubmittedForDispatchedQuery()
                 Optional.empty(),
                 new NodeVersion("test"));
         QueryMonitor queryMonitor = new QueryMonitor(
-                JsonCodec.jsonCodec(StagesInfo.class),
-                JsonCodec.jsonCodec(OperatorStats.class),
-                JsonCodec.jsonCodec(ExecutionFailureInfo.class),
-                JsonCodec.jsonCodec(StatsAndCosts.class),
+                jsonCodec(StagesInfo.class),
+                jsonCodec(OperatorStats.class),
+                jsonCodec(ExecutionFailureInfo.class),
+                jsonCodec(StatsAndCosts.class),
                 new EventListenerManager(new EventListenerConfig(), new SecretsResolver(ImmutableMap.of()), noop(), noopTracer(), new NodeVersion("test")),
                 new NodeInfo("node"),
                 new NodeVersion("version"),

core/trino-main/src/test/java/io/trino/execution/TestColumn.java

@@ -16,11 +16,12 @@
 import io.airlift.json.JsonCodec;
 import org.junit.jupiter.api.Test;
 
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static org.assertj.core.api.Assertions.assertThat;
 
 public class TestColumn
 {
-    private static final JsonCodec<Column> codec = JsonCodec.jsonCodec(Column.class);
+    private static final JsonCodec<Column> codec = jsonCodec(Column.class);
 
     @Test
     public void testRoundTrip()

core/trino-main/src/test/java/io/trino/execution/TestInput.java

@@ -22,11 +22,12 @@
 
 import java.util.Optional;
 
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static org.assertj.core.api.Assertions.assertThat;
 
 public class TestInput
 {
-    private static final JsonCodec<Input> codec = JsonCodec.jsonCodec(Input.class);
+    private static final JsonCodec<Input> codec = jsonCodec(Input.class);
 
     @Test
     public void testRoundTrip()

core/trino-main/src/test/java/io/trino/execution/TestQueryStats.java

@@ -33,6 +33,7 @@
 import java.util.Optional;
 import java.util.OptionalDouble;
 
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static io.airlift.units.DataSize.succinctBytes;
 import static io.trino.server.DynamicFilterService.DynamicFiltersStats;
 import static java.util.concurrent.TimeUnit.NANOSECONDS;
@@ -279,7 +280,7 @@ public class TestQueryStats
     @Test
     public void testJson()
     {
-        JsonCodec<QueryStats> codec = JsonCodec.jsonCodec(QueryStats.class);
+        JsonCodec<QueryStats> codec = jsonCodec(QueryStats.class);
 
         String json = codec.toJson(EXPECTED);
         QueryStats actual = codec.fromJson(json);

core/trino-main/src/test/java/io/trino/execution/TestStageStats.java

@@ -32,6 +32,7 @@
 import java.time.Instant;
 import java.util.Optional;
 
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static java.util.concurrent.TimeUnit.NANOSECONDS;
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -119,7 +120,7 @@ public class TestStageStats
     @Test
     public void testJson()
     {
-        JsonCodec<StageStats> codec = JsonCodec.jsonCodec(StageStats.class);
+        JsonCodec<StageStats> codec = jsonCodec(StageStats.class);
 
         String json = codec.toJson(EXPECTED);
         StageStats actual = codec.fromJson(json);

core/trino-main/src/test/java/io/trino/metadata/TestQualifiedObjectName.java

@@ -16,12 +16,13 @@
 import io.airlift.json.JsonCodec;
 import org.junit.jupiter.api.Test;
 
+import static io.airlift.json.JsonCodec.jsonCodec;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 public class TestQualifiedObjectName
 {
-    private final JsonCodec<QualifiedObjectName> codec = JsonCodec.jsonCodec(QualifiedObjectName.class);
+    private final JsonCodec<QualifiedObjectName> codec = jsonCodec(QualifiedObjectName.class);
 
     @Test
     public void testJsonSerializationRoundTrip()