From d4b8fa17f8612a65f5aa5c659a04015cbb5ebfd3 Mon Sep 17 00:00:00 2001 From: Ihor Andrianov Date: Thu, 3 Apr 2025 22:06:46 +0300 Subject: [PATCH] fix tests --- core/translate/group_by.rs | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/core/translate/group_by.rs b/core/translate/group_by.rs index a36e82864..5adcf658f 100644 --- a/core/translate/group_by.rs +++ b/core/translate/group_by.rs @@ -174,9 +174,6 @@ pub fn emit_group_by<'a>( .sum::(); let group_by_count = group_by.exprs.len(); let non_group_by_non_agg_column_count = non_group_by_non_agg_column_count.unwrap(); - // Count of GROUP BY columns that appear in the result set - let group_by_colls_in_result_set = - plan.result_columns.len() - non_group_by_non_agg_column_count - plan.aggregates.len(); // We have to know which group by expr present in resulting set let group_by_expr_in_res_cols: Vec = group_by @@ -492,10 +489,7 @@ pub fn emit_group_by<'a>( program.emit_insn(Insn::Null { dest: start_reg, dest_end: Some( - start_reg - + non_group_by_non_agg_column_count - + group_by_colls_in_result_set - + plan.aggregates.len() + start_reg + non_group_by_non_agg_column_count + group_by_count + plan.aggregates.len() - 1, ), });