fix(query): fix aggregator_groups_builder to work with string view (#16843)

Author: sundy-li

src/query/service/src/pipelines/processors/transforms/group_by/aggregator_groups_builder.rs

@@ -92,12 +92,16 @@ impl<'a> SerializedKeysGroupColumnsBuilder<'a> {
                         Some(StringColumnBuilder::with_capacity(capacity)),
                         vec![],
                     )
-                } else {
+                } else if params.group_data_types[0].is_binary()
+                    || params.group_data_types[0].is_variant()
+                {
                     (
                         Some(BinaryColumnBuilder::with_capacity(capacity, data_capacity)),
                         None,
                         vec![],
                     )
+                } else {
+                    (None, None, Vec::with_capacity(capacity))
                 }
             } else {
                 (None, None, Vec::with_capacity(capacity))

tests/sqllogictests/suites/base/03_common/03_0003_select_group_by.test

@@ -21,6 +21,13 @@ SELECT number%3 as c1 FROM numbers_mt(10) where number > 2 group by number%3 ord
 1
 2
 
+query I
+SELECT try_cast(number % 3 as string) c1, count()  FROM numbers_mt(10) where number > 2 group by c1 order by c1
+----
+0 3
+1 2
+2 2
+
 query III
 SELECT a,b,count() from (SELECT cast((number%4) AS bigint) as a, cast((number%20) AS bigint) as b from numbers(100)) group by a,b order by a,b limit 3
 ----

tests/sqllogictests/suites/base/03_common/03_0003_select_group_by_old.test

@@ -0,0 +1,7 @@
+statement ok
+set enable_experimental_aggregate_hashtable = 0;
+
+include ./03_0003_select_group_by.test
+
+statement ok
+unset enable_experimental_aggregate_hashtable;