Remove redundant not null check in HttpRemoteTask

Condition `tableScanSplits != null` is always true, because `Multimap.get` returns an empty collection, not null.
trinodb · Jun 16, 2022 · 983488e · 983488e
1 parent 8935a09
commit 983488e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/core/trino-main/src/main/java/io/trino/server/remotetask/HttpRemoteTask.java b/core/trino-main/src/main/java/io/trino/server/remotetask/HttpRemoteTask.java
@@ -248,7 +248,7 @@ public HttpRemoteTask(
             long pendingSourceSplitsWeight = 0;
             for (PlanNodeId planNodeId : planFragment.getPartitionedSources()) {
                 Collection<Split> tableScanSplits = initialSplits.get(planNodeId);
-                if (tableScanSplits != null && !tableScanSplits.isEmpty()) {
+                if (!tableScanSplits.isEmpty()) {
                     pendingSourceSplitCount += tableScanSplits.size();
                     pendingSourceSplitsWeight = addExact(pendingSourceSplitsWeight, SplitWeight.rawValueSum(tableScanSplits, Split::getSplitWeight));
                 }