1
0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-08-07 03:22:57 +03:00

MCOL-3662 Restore error message for multi parameter aggregates save for UDAnF and GROUP_CONCAT

This commit is contained in:
David Hall
2020-01-23 14:10:15 -06:00
parent 011cc70e17
commit 57bf5303f1

View File

@@ -4559,10 +4559,9 @@ ReturnedColumn* buildAggregateColumn(Item* item, gp_walk_info& gwi)
if (gwi.clauseType == SELECT) if (gwi.clauseType == SELECT)
gwi.aggOnSelect = true; gwi.aggOnSelect = true;
// N.B. argument_count() is the # of formal parms to the agg fcn. InifniDB only supports 1 argument // Argument_count() is the # of formal parms to the agg fcn. Columnstore
// TODO: Support more than one parm // only supports 1 argument except UDAnF and GROUP_CONCAT
#if 0 // TODO: Support more than one parm for COUNT(DISTINCT)
if (isp->argument_count() != 1 && isp->sum_func() != Item_sum::GROUP_CONCAT_FUNC if (isp->argument_count() != 1 && isp->sum_func() != Item_sum::GROUP_CONCAT_FUNC
&& isp->sum_func() != Item_sum::UDF_SUM_FUNC) && isp->sum_func() != Item_sum::UDF_SUM_FUNC)
{ {
@@ -4571,7 +4570,6 @@ ReturnedColumn* buildAggregateColumn(Item* item, gp_walk_info& gwi)
return NULL; return NULL;
} }
#endif
AggregateColumn* ac = NULL; AggregateColumn* ac = NULL;
if (isp->sum_func() == Item_sum::GROUP_CONCAT_FUNC) if (isp->sum_func() == Item_sum::GROUP_CONCAT_FUNC)