thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-21 09:10:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixed error with converting types of arrays in IN or VALUES [#METR-21885].

Browse Source
This commit is contained in:
Alexey Milovidov 2016-06-29 00:28:03 +03:00
parent 5ded454986
commit 28b75ba8bc
1 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
dbms/src/Interpreters/convertFieldToType.cpp
View File

@ -125,17 +125,33 @@ Field convertFieldToType(const Field & src, const IDataType & type)
throw Exception("Type mismatch in IN or VALUES section: " + type.getName() + " expected, " throw Exception("Type mismatch in IN or VALUES section: " + type.getName() + " expected, "
+ Field::Types::toString(src.getType()) + " got", ErrorCodes::TYPE_MISMATCH); + Field::Types::toString(src.getType()) + " got", ErrorCodes::TYPE_MISMATCH);
} }
else if (const DataTypeArray * type_array = typeid_cast<const DataTypeArray *>(&type))
{
if (src.getType() != Field::Types::Array)
throw Exception("Type mismatch in IN or VALUES section: " + type.getName() + " expected, "
+ Field::Types::toString(src.getType()) + " got", ErrorCodes::TYPE_MISMATCH);
const IDataType & nested_type = *type_array->getNestedType();
const Array & src_arr = src.get<Array>();
size_t src_arr_size = src_arr.size();
Array res(src_arr_size);
for (size_t i = 0; i < src_arr_size; ++i)
res[i] = convertFieldToType(src_arr[i], nested_type);
return res;
}
else else
{ {
if (src.getType() == Field::Types::UInt64 if (src.getType() == Field::Types::UInt64
|| src.getType() == Field::Types::Int64 || src.getType() == Field::Types::Int64
|| src.getType() == Field::Types::Float64 || src.getType() == Field::Types::Float64
|| src.getType() == Field::Types::Null || src.getType() == Field::Types::Null
|| src.getType() == Field::Types::Array
|| (src.getType() == Field::Types::String || (src.getType() == Field::Types::String
&& !typeid_cast<const DataTypeString *>(&type) && !typeid_cast<const DataTypeString *>(&type)
&& !typeid_cast<const DataTypeFixedString *>(&type)) && !typeid_cast<const DataTypeFixedString *>(&type)))
|| (src.getType() == Field::Types::Array
&& !typeid_cast<const DataTypeArray *>(&type)))
throw Exception("Type mismatch in IN or VALUES section: " + type.getName() + " expected, " throw Exception("Type mismatch in IN or VALUES section: " + type.getName() + " expected, "
+ Field::Types::toString(src.getType()) + " got", ErrorCodes::TYPE_MISMATCH); + Field::Types::toString(src.getType()) + " got", ErrorCodes::TYPE_MISMATCH);
} }