From ac028af817edb3a02729f58614d941e7dde14e07 Mon Sep 17 00:00:00 2001
From: Piotr Findeisen <piotr.findeisen@gmail.com>
Date: Tue, 22 Mar 2022 15:36:20 +0100
Subject: [PATCH] Flatten nested IFs

---
 .../iterative/rule/PushPredicateIntoTableScan.java   | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/iterative/rule/PushPredicateIntoTableScan.java b/core/trino-main/src/main/java/io/trino/sql/planner/iterative/rule/PushPredicateIntoTableScan.java
index a8118e198a43..b67d5d0429e6 100644
--- a/core/trino-main/src/main/java/io/trino/sql/planner/iterative/rule/PushPredicateIntoTableScan.java
+++ b/core/trino-main/src/main/java/io/trino/sql/planner/iterative/rule/PushPredicateIntoTableScan.java
@@ -332,14 +332,12 @@ private static SplitExpression splitExpression(PlannerContext plannerContext, Ex
                 // dynamic filters have no meaning for connectors, so don't pass them
                 dynamicFilters.add(conjunct);
             }
+            else if (isDeterministic(conjunct, metadata)) {
+                deterministicPredicates.add(conjunct);
+            }
             else {
-                if (isDeterministic(conjunct, metadata)) {
-                    deterministicPredicates.add(conjunct);
-                }
-                else {
-                    // don't include non-deterministic predicates
-                    nonDeterministicPredicate.add(conjunct);
-                }
+                // don't include non-deterministic predicates
+                nonDeterministicPredicate.add(conjunct);
             }
         }