Use parenthesis to make boolean logic clearer in QueryPlanner

trinodb · Aug 10, 2023 · 97a1277 · 97a1277
1 parent aee6bc8
commit 97a1277
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/QueryPlanner.java b/core/trino-main/src/main/java/io/trino/sql/planner/QueryPlanner.java
@@ -2127,7 +2127,7 @@ private PlanBuilder distinct(PlanBuilder subPlan, QuerySpecification node, List<
 
     private Optional<OrderingScheme> orderingScheme(PlanBuilder subPlan, Optional<OrderBy> orderBy, List<Expression> orderByExpressions)
     {
-        if (orderBy.isEmpty() || (isSkipRedundantSort(session)) && analysis.isOrderByRedundant(orderBy.get())) {
+        if (orderBy.isEmpty() || (isSkipRedundantSort(session) && analysis.isOrderByRedundant(orderBy.get()))) {
             return Optional.empty();
         }