[Refactor] remove useless function checkCanPrune (backport #52441) (#…

…52444) Co-authored-by: Murphy <[email protected]>
StarRocks · Oct 29, 2024 · dcd3144 · dcd3144
1 parent dea8203
commit dcd3144
Showing 1 changed file with 0 additions and 23 deletions.
diff --git a/fe/fe-core/src/main/java/com/starrocks/sql/optimizer/rule/tree/PruneComplexTypeUtil.java b/fe/fe-core/src/main/java/com/starrocks/sql/optimizer/rule/tree/PruneComplexTypeUtil.java
@@ -103,13 +103,6 @@ public void addAccessPaths(ColumnRefOperator columnRefOperator,
         }
 
         public void add(ColumnRefOperator outputColumnRefOperator, ScalarOperator scalarOperator) {
-            // If outputColumnRefOperator's type is not equal to scalarOperator's type,
-            // we just stop pruning in case.
-            if (!checkCanPrune(outputColumnRefOperator, scalarOperator)) {
-                setEnablePruneComplexTypes(false);
-                return;
-            }
-
             ComplexTypeAccessGroup visitedAccessGroup = null;
             if (outputColumnRefOperator != null) {
                 // If outputColumnRefOperator is not null, it means it may have visited access group,
@@ -158,22 +151,6 @@ private ColumnRefOperator getOriginalColRef(ColumnRefOperator col) {
             return col;
         }
 
-        private boolean checkCanPrune(ColumnRefOperator columnRefOperator, ScalarOperator scalarOperator) {
-            if (columnRefOperator == null) {
-                return true;
-            }
-
-            if (!columnRefOperator.getType().isComplexType() && !scalarOperator.getType().isComplexType()) {
-                // If columnRefOperator and scalarOperator both are not complex type, don't need to check
-                return true;
-            }
-
-            if (!columnRefOperator.getType().equals(scalarOperator.getType())) {
-                LOG.warn(String.format("Complex type columnRefOperator[%s] and scalarOperator[%s] should has the same " +
-                        "type", columnRefOperator.getType(), scalarOperator.getType()));
-            }
-            return true;
-        }
     }
 
     private static ComplexTypeAccessPaths concatAccessPaths(