Remove redundant else

trinodb · Aug 10, 2022 · 92f797d · 92f797d
1 parent 1156d0c
commit 92f797d
Showing 1 changed file with 6 additions and 18 deletions.
diff --git a/core/trino-main/src/main/java/io/trino/sql/planner/DomainTranslator.java b/core/trino-main/src/main/java/io/trino/sql/planner/DomainTranslator.java
@@ -683,45 +683,33 @@ the Domain should consist of ranges (which do not sum to the whole ValueSet), an
                     if (complement) {
                         return Optional.empty();
                     }
-                    else {
-                        return Optional.of(Domain.create(ValueSet.ofRanges(Range.equal(type, value)), false));
-                    }
+                    return Optional.of(Domain.create(ValueSet.ofRanges(Range.equal(type, value)), false));
                 case GREATER_THAN:
                     if (complement) {
                         return Optional.empty();
                     }
-                    else {
-                        return Optional.of(Domain.create(ValueSet.ofRanges(Range.greaterThan(type, value)), false));
-                    }
+                    return Optional.of(Domain.create(ValueSet.ofRanges(Range.greaterThan(type, value)), false));
                 case GREATER_THAN_OR_EQUAL:
                     if (complement) {
                         return Optional.empty();
                     }
-                    else {
-                        return Optional.of(Domain.create(ValueSet.ofRanges(Range.greaterThanOrEqual(type, value)), false));
-                    }
+                    return Optional.of(Domain.create(ValueSet.ofRanges(Range.greaterThanOrEqual(type, value)), false));
                 case LESS_THAN:
                     if (complement) {
                         return Optional.empty();
                     }
-                    else {
-                        return Optional.of(Domain.create(ValueSet.ofRanges(Range.lessThan(type, value)), false));
-                    }
+                    return Optional.of(Domain.create(ValueSet.ofRanges(Range.lessThan(type, value)), false));
                 case LESS_THAN_OR_EQUAL:
                     if (complement) {
                         return Optional.empty();
                     }
-                    else {
-                        return Optional.of(Domain.create(ValueSet.ofRanges(Range.lessThanOrEqual(type, value)), false));
-                    }
+                    return Optional.of(Domain.create(ValueSet.ofRanges(Range.lessThanOrEqual(type, value)), false));
                 case NOT_EQUAL:
                 case IS_DISTINCT_FROM:
                     if (complement) {
                         return Optional.of(Domain.create(ValueSet.ofRanges(Range.equal(type, value)), false));
                     }
-                    else {
-                        return Optional.empty();
-                    }
+                    return Optional.empty();
             }
             throw new AssertionError("Unhandled operator: " + comparisonOperator);
         }