Skip to content

Commit f89f8b7

Browse files
authored
Merge pull request ClickHouse#63619 from yohannj/sumMapErrorMsg
harmonize sumMap error messages on ILLEGAL_TYPE_OF_ARGUMENT
2 parents 6247f6a + d3ff638 commit f89f8b7

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/AggregateFunctions/AggregateFunctionSumMap.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -711,7 +711,7 @@ auto parseArguments(const std::string & name, const DataTypes & arguments)
711711

712712
const auto * array_type = checkAndGetDataType<DataTypeArray>(args[0].get());
713713
if (!array_type)
714-
throw Exception(ErrorCodes::ILLEGAL_TYPE_OF_ARGUMENT, "First argument for function {} must be an array, not {}",
714+
throw Exception(ErrorCodes::ILLEGAL_TYPE_OF_ARGUMENT, "Argument #1 for function {} must be an array, not {}",
715715
name, args[0]->getName());
716716

717717
DataTypePtr keys_type = array_type->getNestedType();
@@ -722,8 +722,8 @@ auto parseArguments(const std::string & name, const DataTypes & arguments)
722722
{
723723
array_type = checkAndGetDataType<DataTypeArray>(args[i].get());
724724
if (!array_type)
725-
throw Exception(ErrorCodes::ILLEGAL_TYPE_OF_ARGUMENT, "Argument #{} for function {} must be an array.",
726-
i, name);
725+
throw Exception(ErrorCodes::ILLEGAL_TYPE_OF_ARGUMENT, "Argument #{} for function {} must be an array, not {}",
726+
i + 1, name, args[i]->getName());
727727
values_types.push_back(array_type->getNestedType());
728728
}
729729

0 commit comments

Comments
 (0)