Consistent empty braces formatting

trinodb · Oct 30, 2024 · 3f66eea · 3f66eea
1 parent 61fd8c1
commit 3f66eea
Show file tree

Hide file tree

Showing 159 changed files with 223 additions and 668 deletions.
diff --git a/client/trino-cli/src/main/java/io/trino/cli/NullPrinter.java b/client/trino-cli/src/main/java/io/trino/cli/NullPrinter.java
@@ -19,12 +19,8 @@ public class NullPrinter
         implements OutputPrinter
 {
     @Override
-    public void printRows(List<List<?>> rows, boolean complete)
-    {
-    }
+    public void printRows(List<List<?>> rows, boolean complete) {}
 
     @Override
-    public void finish()
-    {
-    }
+    public void finish() {}
 }
diff --git a/client/trino-client/src/main/java/io/trino/client/auth/external/MemoryCachedKnownToken.java b/client/trino-client/src/main/java/io/trino/client/auth/external/MemoryCachedKnownToken.java
@@ -43,9 +43,7 @@ class MemoryCachedKnownToken
     private final Lock writeLock = lock.writeLock();
     private Optional<Token> knownToken = Optional.empty();
 
-    private MemoryCachedKnownToken()
-    {
-    }
+    private MemoryCachedKnownToken() {}
 
     @Override
     public Optional<Token> getToken()

diff --git a/client/trino-client/src/test/java/io/trino/client/auth/external/MockTokenPoller.java b/client/trino-client/src/test/java/io/trino/client/auth/external/MockTokenPoller.java
@@ -40,9 +40,7 @@ public TokenPollResult pollForToken(URI tokenUri, Duration timeout)
             }
 
             @Override
-            public void tokenReceived(URI tokenUri)
-            {
-            }
+            public void tokenReceived(URI tokenUri) {}
         };
     }
 

diff --git a/client/trino-jdbc/src/main/java/io/trino/jdbc/TestingRedirectHandlerInjector.java b/client/trino-jdbc/src/main/java/io/trino/jdbc/TestingRedirectHandlerInjector.java
@@ -18,9 +18,7 @@
 
 public class TestingRedirectHandlerInjector
 {
-    private TestingRedirectHandlerInjector()
-    {
-    }
+    private TestingRedirectHandlerInjector() {}
 
     public static void setRedirectHandler(Consumer<URI> redirectHandler)
     {

diff --git a/core/trino-main/src/main/java/io/trino/TrinoMediaTypes.java b/core/trino-main/src/main/java/io/trino/TrinoMediaTypes.java
@@ -20,7 +20,5 @@ public final class TrinoMediaTypes
     public static final String TRINO_PAGES = "application/x-trino-pages";
     public static final MediaType TRINO_PAGES_TYPE = MediaType.create("application", "x-trino-pages");
 
-    private TrinoMediaTypes()
-    {
-    }
+    private TrinoMediaTypes() {}
 }
diff --git a/core/trino-main/src/main/java/io/trino/block/BlockSerdeUtil.java b/core/trino-main/src/main/java/io/trino/block/BlockSerdeUtil.java
@@ -29,9 +29,7 @@ public final class BlockSerdeUtil
     public static final MethodHandle READ_BLOCK = methodHandle(BlockSerdeUtil.class, "readBlock", BlockEncodingSerde.class, Slice.class);
     public static final MethodHandle READ_BLOCK_VALUE = methodHandle(BlockSerdeUtil.class, "readBlockValue", BlockEncodingSerde.class, Type.class, Slice.class);
 
-    private BlockSerdeUtil()
-    {
-    }
+    private BlockSerdeUtil() {}
 
     public static Block readBlock(BlockEncodingSerde blockEncodingSerde, Slice slice)
     {

diff --git a/core/trino-main/src/main/java/io/trino/execution/NoOpFailureInjector.java b/core/trino-main/src/main/java/io/trino/execution/NoOpFailureInjector.java
@@ -22,9 +22,7 @@ public class NoOpFailureInjector
         implements FailureInjector
 {
     @Override
-    public void injectTaskFailure(String traceToken, int stageId, int partitionId, int attemptId, InjectedFailureType injectionType, Optional<ErrorType> errorType)
-    {
-    }
+    public void injectTaskFailure(String traceToken, int stageId, int partitionId, int attemptId, InjectedFailureType injectionType, Optional<ErrorType> errorType) {}
 
     @Override
     public Optional<InjectedFailure> getInjectedFailure(String traceToken, int stageId, int partitionId, int attemptId)

diff --git a/...rino-main/src/main/java/io/trino/execution/scheduler/FixedSourcePartitionedScheduler.java b/...rino-main/src/main/java/io/trino/execution/scheduler/FixedSourcePartitionedScheduler.java
@@ -193,9 +193,7 @@ public SplitPlacementResult computeAssignments(Set<Split> splits)
         }
 
         @Override
-        public void lockDownNodes()
-        {
-        }
+        public void lockDownNodes() {}
 
         @Override
         public List<InternalNode> allNodes()

diff --git a/core/trino-main/src/main/java/io/trino/execution/scheduler/TaskLifecycleListener.java b/core/trino-main/src/main/java/io/trino/execution/scheduler/TaskLifecycleListener.java
@@ -25,13 +25,9 @@ public interface TaskLifecycleListener
     TaskLifecycleListener NO_OP = new TaskLifecycleListener()
     {
         @Override
-        public void taskCreated(PlanFragmentId fragmentId, RemoteTask task)
-        {
-        }
+        public void taskCreated(PlanFragmentId fragmentId, RemoteTask task) {}
 
         @Override
-        public void noMoreTasks(PlanFragmentId fragmentId)
-        {
-        }
+        public void noMoreTasks(PlanFragmentId fragmentId) {}
     };
 }
diff --git a/core/trino-main/src/main/java/io/trino/operator/DeduplicatingDirectExchangeBuffer.java b/core/trino-main/src/main/java/io/trino/operator/DeduplicatingDirectExchangeBuffer.java
@@ -803,9 +803,7 @@ public long getRetainedSizeInBytes()
         }
 
         @Override
-        public void close()
-        {
-        }
+        public void close() {}
     }
 
     @NotThreadSafe

diff --git a/core/trino-main/src/main/java/io/trino/operator/DevNullOperator.java b/core/trino-main/src/main/java/io/trino/operator/DevNullOperator.java
@@ -40,9 +40,7 @@ public Operator createOperator(DriverContext driverContext)
         }
 
         @Override
-        public void noMoreOperators()
-        {
-        }
+        public void noMoreOperators() {}
 
         @Override
         public OperatorFactory duplicate()
@@ -72,9 +70,7 @@ public boolean needsInput()
     }
 
     @Override
-    public void addInput(Page page)
-    {
-    }
+    public void addInput(Page page) {}
 
     @Override
     public Page getOutput()

diff --git a/core/trino-main/src/main/java/io/trino/operator/FinishedOperator.java b/core/trino-main/src/main/java/io/trino/operator/FinishedOperator.java
@@ -34,9 +34,7 @@ public OperatorContext getOperatorContext()
     }
 
     @Override
-    public void finish()
-    {
-    }
+    public void finish() {}
 
     @Override
     public boolean isFinished()

diff --git a/core/trino-main/src/main/java/io/trino/operator/Operator.java b/core/trino-main/src/main/java/io/trino/operator/Operator.java
@@ -78,9 +78,7 @@ default ListenableFuture<Void> startMemoryRevoke()
      * Clean up and release resources after completed memory revoking. Called by driver
      * once future returned by startMemoryRevoke is completed.
      */
-    default void finishMemoryRevoke()
-    {
-    }
+    default void finishMemoryRevoke() {}
 
     /**
      * Notifies the operator that no more pages will be added and the

diff --git a/core/trino-main/src/main/java/io/trino/operator/PositionSearcher.java b/core/trino-main/src/main/java/io/trino/operator/PositionSearcher.java
@@ -19,9 +19,7 @@
 
 public class PositionSearcher
 {
-    private PositionSearcher()
-    {
-    }
+    private PositionSearcher() {}
 
     /**
      * @param startPosition - inclusive

diff --git a/core/trino-main/src/main/java/io/trino/operator/RefreshMaterializedViewOperator.java b/core/trino-main/src/main/java/io/trino/operator/RefreshMaterializedViewOperator.java
@@ -114,9 +114,7 @@ public Page getOutput()
     }
 
     @Override
-    public void finish()
-    {
-    }
+    public void finish() {}
 
     @Override
     public void close()

diff --git a/core/trino-main/src/main/java/io/trino/operator/aggregation/DoubleHistogramAggregation.java b/core/trino-main/src/main/java/io/trino/operator/aggregation/DoubleHistogramAggregation.java
@@ -39,9 +39,7 @@ public final class DoubleHistogramAggregation
 {
     public static final int ENTRY_BUFFER_SIZE = 100;
 
-    private DoubleHistogramAggregation()
-    {
-    }
+    private DoubleHistogramAggregation() {}
 
     @AccumulatorStateMetadata(stateSerializerClass = DoubleHistogramStateSerializer.class, stateFactoryClass = NumericHistogramStateFactory.class)
     public interface State

diff --git a/...rino-main/src/main/java/io/trino/operator/aggregation/partial/SkipAggregationBuilder.java b/...rino-main/src/main/java/io/trino/operator/aggregation/partial/SkipAggregationBuilder.java
@@ -97,9 +97,7 @@ public void updateMemory()
     }
 
     @Override
-    public void close()
-    {
-    }
+    public void close() {}
 
     @Override
     public ListenableFuture<Void> startMemoryRevoke()

diff --git a/core/trino-main/src/main/java/io/trino/operator/function/RegularTableFunctionPartition.java b/core/trino-main/src/main/java/io/trino/operator/function/RegularTableFunctionPartition.java
@@ -329,9 +329,7 @@ private int findFirstNullPosition(int markerChannel)
         return end;
     }
 
-    public record PassThroughColumnSpecification(boolean isPartitioningColumn, int inputChannel, int indexChannel)
-    {
-    }
+    public record PassThroughColumnSpecification(boolean isPartitioningColumn, int inputChannel, int indexChannel) {}
 
     private PassThroughColumnProvider createColumnProvider(PassThroughColumnSpecification specification)
     {

diff --git a/core/trino-main/src/main/java/io/trino/operator/index/IndexLoader.java b/core/trino-main/src/main/java/io/trino/operator/index/IndexLoader.java
@@ -428,8 +428,6 @@ public void appendTo(long position, PageBuilder pageBuilder, int outputChannelOf
         }
 
         @Override
-        public void close()
-        {
-        }
+        public void close() {}
     }
 }
diff --git a/core/trino-main/src/main/java/io/trino/operator/index/IndexSnapshot.java b/core/trino-main/src/main/java/io/trino/operator/index/IndexSnapshot.java
@@ -61,7 +61,5 @@ public void appendTo(long position, PageBuilder pageBuilder, int outputChannelOf
     }
 
     @Override
-    public void close()
-    {
-    }
+    public void close() {}
 }
diff --git a/core/trino-main/src/main/java/io/trino/operator/index/PageBufferOperator.java b/core/trino-main/src/main/java/io/trino/operator/index/PageBufferOperator.java
@@ -51,9 +51,7 @@ public Operator createOperator(DriverContext driverContext)
         }
 
         @Override
-        public void noMoreOperators()
-        {
-        }
+        public void noMoreOperators() {}
 
         @Override
         public OperatorFactory duplicate()

diff --git a/core/trino-main/src/main/java/io/trino/operator/index/PageRecordSet.java b/core/trino-main/src/main/java/io/trino/operator/index/PageRecordSet.java
@@ -144,8 +144,6 @@ public boolean isNull(int field)
         }
 
         @Override
-        public void close()
-        {
-        }
+        public void close() {}
     }
 }
diff --git a/core/trino-main/src/main/java/io/trino/operator/join/JoinHash.java b/core/trino-main/src/main/java/io/trino/operator/join/JoinHash.java
@@ -163,7 +163,5 @@ public void appendTo(long position, PageBuilder pageBuilder, int outputChannelOf
     }
 
     @Override
-    public void close()
-    {
-    }
+    public void close() {}
 }
diff --git a/core/trino-main/src/main/java/io/trino/operator/join/PartitionedLookupSourceFactory.java b/core/trino-main/src/main/java/io/trino/operator/join/PartitionedLookupSourceFactory.java
@@ -571,9 +571,7 @@ public boolean isJoinPositionEligible(long currentJoinPosition, int probePositio
         }
 
         @Override
-        public void close()
-        {
-        }
+        public void close() {}
     }
 
     @Immutable

diff --git a/core/trino-main/src/main/java/io/trino/operator/output/TaskOutputOperator.java b/core/trino-main/src/main/java/io/trino/operator/output/TaskOutputOperator.java
@@ -83,9 +83,7 @@ public Operator createOperator(DriverContext driverContext)
         }
 
         @Override
-        public void noMoreOperators()
-        {
-        }
+        public void noMoreOperators() {}
 
         @Override
         public OperatorFactory duplicate()

diff --git a/core/trino-main/src/main/java/io/trino/operator/scalar/JsonOperators.java b/core/trino-main/src/main/java/io/trino/operator/scalar/JsonOperators.java
@@ -62,9 +62,7 @@ public final class JsonOperators
 {
     private static final JsonFactory JSON_FACTORY = createJsonFactory();
 
-    private JsonOperators()
-    {
-    }
+    private JsonOperators() {}
 
     @ScalarOperator(CAST)
     @SqlNullable

diff --git a/core/trino-main/src/main/java/io/trino/operator/table/json/execution/ExecutionPlanner.java b/core/trino-main/src/main/java/io/trino/operator/table/json/execution/ExecutionPlanner.java
@@ -41,9 +41,7 @@
 
 public class ExecutionPlanner
 {
-    private ExecutionPlanner()
-    {
-    }
+    private ExecutionPlanner() {}
 
     public static JsonTableProcessingFragment getExecutionPlan(
             JsonTablePlanNode plan,

diff --git a/core/trino-main/src/main/java/io/trino/operator/table/json/execution/SequenceEvaluator.java b/core/trino-main/src/main/java/io/trino/operator/table/json/execution/SequenceEvaluator.java
@@ -30,9 +30,7 @@
 
 public class SequenceEvaluator
 {
-    private SequenceEvaluator()
-    {
-    }
+    private SequenceEvaluator() {}
 
     // creates a sequence of JSON items, and applies error handling
     public static List<JsonNode> getSequence(JsonNode item, Object[] pathParameters, JsonPathEvaluator pathEvaluator, boolean errorOnError)

diff --git a/core/trino-main/src/main/java/io/trino/sql/gen/Bootstrap.java b/core/trino-main/src/main/java/io/trino/sql/gen/Bootstrap.java
@@ -37,9 +37,7 @@ public final class Bootstrap
         }
     }
 
-    private Bootstrap()
-    {
-    }
+    private Bootstrap() {}
 
     public static CallSite bootstrap(MethodHandles.Lookup callerLookup, String name, MethodType type, long bindingId)
     {

diff --git a/core/trino-main/src/main/java/io/trino/sql/gen/LambdaCapture.java b/core/trino-main/src/main/java/io/trino/sql/gen/LambdaCapture.java
@@ -34,9 +34,7 @@ public final class LambdaCapture
         }
     }
 
-    private LambdaCapture()
-    {
-    }
+    private LambdaCapture() {}
 
     public static CallSite lambdaCapture(
             MethodHandles.Lookup callerLookup,

diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/ExpressionExtractor.java b/core/trino-main/src/main/java/io/trino/sql/planner/ExpressionExtractor.java
@@ -60,9 +60,7 @@ public static void forEachExpression(PlanNode plan, Consumer<Expression> express
         plan.accept(new Visitor(expressionConsumer, true, noLookup()), null);
     }
 
-    private ExpressionExtractor()
-    {
-    }
+    private ExpressionExtractor() {}
 
     private static class Visitor
             extends SimplePlanVisitor<Void>

diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/IrTypeAnalyzer.java b/core/trino-main/src/main/java/io/trino/sql/planner/IrTypeAnalyzer.java
@@ -20,7 +20,5 @@
  */
 public class IrTypeAnalyzer
 {
-    public IrTypeAnalyzer()
-    {
-    }
+    public IrTypeAnalyzer() {}
 }
diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/iterative/rule/Util.java b/core/trino-main/src/main/java/io/trino/sql/planner/iterative/rule/Util.java
@@ -45,9 +45,7 @@ final class Util
     private static final CatalogSchemaFunctionName ROW_NUMBER_NAME = builtinFunctionName("row_number");
     private static final CatalogSchemaFunctionName RANK_NAME = builtinFunctionName("rank");
 
-    private Util()
-    {
-    }
+    private Util() {}
 
     /**
      * Prune the set of available inputs to those required by the given expressions.

diff --git a/core/trino-main/src/main/java/io/trino/testing/NullOutputOperator.java b/core/trino-main/src/main/java/io/trino/testing/NullOutputOperator.java
@@ -61,9 +61,7 @@ public Operator createOperator(DriverContext driverContext)
         }
 
         @Override
-        public void noMoreOperators()
-        {
-        }
+        public void noMoreOperators() {}
 
         @Override
         public OperatorFactory duplicate()

diff --git a/core/trino-main/src/main/java/io/trino/type/ArrayParametricType.java b/core/trino-main/src/main/java/io/trino/type/ArrayParametricType.java
@@ -30,9 +30,7 @@ public final class ArrayParametricType
 {
     public static final ArrayParametricType ARRAY = new ArrayParametricType();
 
-    private ArrayParametricType()
-    {
-    }
+    private ArrayParametricType() {}
 
     @Override
     public String getName()