diff --git a/presto-main/src/main/java/io/prestosql/sql/planner/EqualityInference.java b/presto-main/src/main/java/io/prestosql/sql/planner/EqualityInference.java index d9317d561e273..be07c365424ae 100644 --- a/presto-main/src/main/java/io/prestosql/sql/planner/EqualityInference.java +++ b/presto-main/src/main/java/io/prestosql/sql/planner/EqualityInference.java @@ -38,7 +38,6 @@ import java.util.Set; import java.util.stream.Collectors; -import static com.google.common.base.Preconditions.checkArgument; import static com.google.common.base.Predicates.equalTo; import static com.google.common.base.Predicates.not; import static com.google.common.collect.Iterables.filter; @@ -81,20 +80,8 @@ private EqualityInference(Multimap equalitySets, Map symbolScope) - { - checkArgument(isDeterministic(expression), "Only deterministic expressions may be considered for rewrite"); - return rewriteExpression(expression, symbolScope, true); - } - - /** - * Attempts to rewrite an Expression in terms of the symbols allowed by the symbol scope - * given the known equalities. Returns null if unsuccessful. - * This method allows rewriting non-deterministic expressions. - */ - public Expression rewriteExpressionAllowNonDeterministic(Expression expression, Predicate symbolScope) { return rewriteExpression(expression, symbolScope, true); } diff --git a/presto-main/src/main/java/io/prestosql/sql/planner/optimizations/PredicatePushDown.java b/presto-main/src/main/java/io/prestosql/sql/planner/optimizations/PredicatePushDown.java index 5dbf4666b2c51..db724742cca03 100644 --- a/presto-main/src/main/java/io/prestosql/sql/planner/optimizations/PredicatePushDown.java +++ b/presto-main/src/main/java/io/prestosql/sql/planner/optimizations/PredicatePushDown.java @@ -1063,7 +1063,7 @@ private PlanNode visitNonFilteringSemiJoin(SemiJoinNode node, RewriteContext