From b8c6cdeec122d63f673f2968de55a7a8ff8123c7 Mon Sep 17 00:00:00 2001 From: zclllyybb Date: Wed, 12 Feb 2025 20:14:10 +0800 Subject: [PATCH] [Fix](agg) Remove useless DCHECK for streaming agg node (#47842) It may be wrong for legacy planner --- be/src/pipeline/exec/distinct_streaming_aggregation_operator.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/be/src/pipeline/exec/distinct_streaming_aggregation_operator.cpp b/be/src/pipeline/exec/distinct_streaming_aggregation_operator.cpp index bb282fd118e5c0..09edac3e3a887e 100644 --- a/be/src/pipeline/exec/distinct_streaming_aggregation_operator.cpp +++ b/be/src/pipeline/exec/distinct_streaming_aggregation_operator.cpp @@ -332,7 +332,6 @@ DistinctStreamingAggOperatorX::DistinctStreamingAggOperatorX(ObjectPool* pool, i _is_streaming_preagg = tnode.agg_node.use_streaming_preaggregation; if (_is_streaming_preagg) { DCHECK(!tnode.agg_node.grouping_exprs.empty()) << "Streaming preaggs do grouping"; - DCHECK(_limit == -1) << "Preaggs have no limits"; } } else { _is_streaming_preagg = false;