Remove implicit fall through.

kgpai · Mar 14, 2024 · 220dcba · 220dcba
1 parent 399345d
commit 220dcba
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/velox/functions/sparksql/aggregates/SumAggregate.cpp b/velox/functions/sparksql/aggregates/SumAggregate.cpp
@@ -113,12 +113,11 @@ exec::AggregateRegistrationResult registerSum(
                 BIGINT());
           }
           case TypeKind::HUGEINT: {
-            if (inputType->isLongDecimal()) {
-              // If inputType is long decimal,
-              // its output type is always long decimal.
-              return std::make_unique<exec::SimpleAggregateAdapter<
-                  DecimalSumAggregate<int128_t, int128_t>>>(resultType);
-            }
+            VELOX_CHECK(inputType->isLongDecimal());
+            // If inputType is long decimal,
+            // its output type is always long decimal.
+            return std::make_unique<exec::SimpleAggregateAdapter<
+                DecimalSumAggregate<int128_t, int128_t>>>(resultType);
           }
           case TypeKind::REAL:
             if (resultType->kind() == TypeKind::REAL) {