Consistent empty braces formatting

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() {}
 }

client/trino-cli/src/main/java/io/trino/cli/QueryPreprocessor.java

@@ -129,6 +129,7 @@ private static String preprocessQueryInternal(Optional<String> catalog, Optional
                             CharStreams.copy(new InputStreamReader(inputStream, UTF_8), builder);
                         }
                         catch (IOException | RuntimeException ignored) {
+                            // ignore
                         }
                         return builder.toString();
                     });
@@ -175,6 +176,7 @@ private static String preprocessQueryInternal(Optional<String> catalog, Optional
                     message = "\n===\n" + message + "\n===";
                 }
                 catch (Exception ignored) {
+                    // ignore
                 }
                 throw new QueryPreprocessorException("Query preprocessor exited " + exitCode + message);
             }

client/trino-client/src/main/java/io/trino/client/OkHttpUtil.java

@@ -323,6 +323,7 @@ private static KeyStore loadTrustStore(File trustStorePath, Optional<String> tru
             }
         }
         catch (IOException | GeneralSecurityException ignored) {
+            // ignore
         }
 
         try (InputStream in = new FileInputStream(trustStorePath)) {

client/trino-client/src/main/java/io/trino/client/PemReader.java

@@ -234,6 +234,7 @@ else if ("ENCRYPTED".equals(keyType)) {
                 return keyFactory.generatePrivate(encodedKeySpec);
             }
             catch (InvalidKeySpecException ignore) {
+                // ignore
             }
         }
         throw new InvalidKeySpecException("Key type must be one of " + algorithms);

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()

client/trino-client/src/main/java/io/trino/client/auth/kerberos/SpnegoHandler.java

@@ -122,6 +122,7 @@ private byte[] generateToken(String servicePrincipal)
                 }
             }
             catch (GSSException ignored) {
+                // ignore
             }
         }
     }

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) {}
         };
     }
 

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)
     {

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() {}
 }

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)
     {

core/trino-main/src/main/java/io/trino/connector/ConnectorServices.java

@@ -126,6 +126,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             splitManager = connector.getSplitManager();
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
         this.splitManager = Optional.ofNullable(splitManager);
 
@@ -135,6 +136,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             requireNonNull(connectorPageSourceProviderFactory, format("Connector '%s' returned a null page source provider factory", catalogHandle));
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
 
         try {
@@ -145,6 +147,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             connectorPageSourceProviderFactory = () -> pageSourceProvider;
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
         this.pageSourceProviderFactory = Optional.ofNullable(connectorPageSourceProviderFactory);
 
@@ -154,6 +157,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             requireNonNull(connectorPageSinkProvider, format("Connector '%s' returned a null page sink provider", catalogHandle));
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
         this.pageSinkProvider = Optional.ofNullable(connectorPageSinkProvider);
 
@@ -163,6 +167,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             requireNonNull(indexProvider, format("Connector '%s' returned a null index provider", catalogHandle));
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
         this.indexProvider = Optional.ofNullable(indexProvider);
 
@@ -172,6 +177,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             requireNonNull(partitioningProvider, format("Connector '%s' returned a null partitioning provider", catalogHandle));
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
         this.partitioningProvider = Optional.ofNullable(partitioningProvider);
 
@@ -180,6 +186,7 @@ public ConnectorServices(Tracer tracer, CatalogHandle catalogHandle, Connector c
             accessControl = connector.getAccessControl();
         }
         catch (UnsupportedOperationException _) {
+            // ignore
         }
         verifyAccessControl(accessControl);
         this.accessControl = Optional.ofNullable(accessControl);
@@ -405,6 +412,7 @@ private static void mustNotDeclareMethod(Class<?> clazz, String name, Class<?>..
                     name, Arrays.stream(parameterTypes).map(Class::getName).collect(Collectors.joining(", "))));
         }
         catch (ReflectiveOperationException _) {
+            // ignore
         }
     }
 }

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)

core/trino-main/src/main/java/io/trino/execution/executor/scheduler/TaskControl.java

@@ -157,6 +157,7 @@ public boolean awaitReady()
                     wakeup.await();
                 }
                 catch (InterruptedException e) {
+                    // ignore
                 }
             }
 
@@ -199,6 +200,7 @@ public void awaitUnblock()
                     wakeup.await();
                 }
                 catch (InterruptedException e) {
+                    // ignore
                 }
             }
         }

core/trino-main/src/main/java/io/trino/execution/executor/timesharing/TimeSharingTaskExecutor.java

@@ -265,6 +265,7 @@ private synchronized void addRunnerThread()
             executor.execute(versionEmbedder.embedVersion(new TaskRunner()));
         }
         catch (RejectedExecutionException _) {
+            // ignore
         }
     }
 

core/trino-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()

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) {}
     };
 }

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

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()

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()

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

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

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()

core/trino-main/src/main/java/io/trino/operator/aggregation/AccumulatorCompiler.java

@@ -329,6 +329,7 @@ private static Function<List<Supplier<Object>>, WindowAccumulator> createWindowA
             };
         }
         catch (ReflectiveOperationException _) {
+            // ignore
         }
 
         try {
@@ -343,6 +344,7 @@ private static Function<List<Supplier<Object>>, WindowAccumulator> createWindowA
             };
         }
         catch (ReflectiveOperationException _) {
+            // ignore
         }
         throw new RuntimeException(format("Window accumulator class %s does not have a constructor with a single List argument or a no-argument constructor", windowAccumulator.getName()));
     }

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

core/trino-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()

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)
     {

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() {}
     }
 }

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() {}
 }

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()

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() {}
     }
 }