Use Collection.isEmpty

trinodb · Jan 11, 2024 · 373aa94 · 373aa94
1 parent e61fd89
commit 373aa94
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/plugin/trino-hive/src/main/java/io/trino/plugin/hive/ReaderProjectionsAdapter.java b/plugin/trino-hive/src/main/java/io/trino/plugin/hive/ReaderProjectionsAdapter.java
@@ -89,7 +89,7 @@ public Page adaptPage(@Nullable Page input)
 
     private static Block createAdaptedLazyBlock(Block inputBlock, List<Integer> dereferenceSequence)
     {
-        if (dereferenceSequence.size() == 0) {
+        if (dereferenceSequence.isEmpty()) {
             return inputBlock;
         }
 
@@ -129,7 +129,7 @@ public Block load()
          */
         private Block loadInternalBlock(List<Integer> dereferences, Block parentBlock)
         {
-            if (dereferences.size() == 0) {
+            if (dereferences.isEmpty()) {
                 return parentBlock.getLoadedBlock();
             }
 

diff --git a/plugin/trino-hive/src/main/java/io/trino/plugin/hive/util/AsyncQueue.java b/plugin/trino-hive/src/main/java/io/trino/plugin/hive/util/AsyncQueue.java
@@ -67,7 +67,7 @@ public AsyncQueue(int targetQueueSize, Executor executor)
      */
     public synchronized boolean isFinished()
     {
-        return finishing && borrowerCount == 0 && elements.size() == 0;
+        return finishing && borrowerCount == 0 && elements.isEmpty();
     }
 
     public synchronized void finish()
@@ -83,7 +83,7 @@ public synchronized void finish()
     private synchronized void signalIfFinishing()
     {
         if (finishing && borrowerCount == 0) {
-            if (elements.size() == 0) {
+            if (elements.isEmpty()) {
                 // Reset elements queue after finishing to avoid holding on to the full sized empty array inside
                 elements = new ArrayDeque<>(0);
                 completeAsync(executor, notEmptySignal);

diff --git a/plugin/trino-hive/src/test/java/io/trino/plugin/hive/TestHiveReaderProjectionsUtil.java b/plugin/trino-hive/src/test/java/io/trino/plugin/hive/TestHiveReaderProjectionsUtil.java
@@ -64,7 +64,7 @@ public static HiveColumnHandle createProjectedColumnHandle(HiveColumnHandle colu
     {
         checkArgument(column.isBaseColumn(), "base column is expected here");
 
-        if (indices.size() == 0) {
+        if (indices.isEmpty()) {
             return column;
         }
 

diff --git a/plugin/trino-hive/src/test/java/io/trino/plugin/hive/TestReaderProjectionsAdapter.java b/plugin/trino-hive/src/test/java/io/trino/plugin/hive/TestReaderProjectionsAdapter.java
@@ -224,7 +224,7 @@ private static void verifyBlock(Block actualBlock, Type outputType, Block input,
 
     private static Block createProjectedColumnBlock(Block data, Type finalType, RowType blockType, List<Integer> dereferences)
     {
-        if (dereferences.size() == 0) {
+        if (dereferences.isEmpty()) {
             return data;
         }