This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push: new 336131a Fix theta-sketch missing break in switch statement (#5811) 336131a is described below commit 336131a32fda71787dec1d5c0b395e02bf3f42ea Author: Xiaotian (Jackie) Jiang <17555551+jackie-ji...@users.noreply.github.com> AuthorDate: Wed Aug 5 10:14:51 2020 -0700 Fix theta-sketch missing break in switch statement (#5811) --- .../function/DistinctCountThetaSketchAggregationFunction.java | 1 + 1 file changed, 1 insertion(+) diff --git a/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java b/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java index 99cef39..d598d3a 100644 --- a/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java +++ b/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountThetaSketchAggregationFunction.java @@ -300,6 +300,7 @@ public class DistinctCountThetaSketchAggregationFunction implements AggregationF getUnionMap(groupByResultHolder, groupKeyArray[i]).get(predicate).update(sketches[i]); } } + break; default: throw new IllegalStateException(); } --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org