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

Don't treat Bool and number variants as suspicious

Browse Source
This commit is contained in:
avogar 2024-03-27 19:19:08 +00:00
parent c21c67f9f5
commit 043c625898
3 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Interpreters/parseColumnsListForTableFunction.cpp
View File

@ -86,6 +86,10 @@ void validateDataType(const DataTypePtr & type_to_check, const DataTypeValidatio
{
for (size_t j = i + 1; j < variants.size(); ++j)
{
/// Don't consider bool as similar to something (like number).
if (isBool(variants[i]) || isBool(variants[j]))
continue;
if (auto supertype = tryGetLeastSupertype(DataTypes{variants[i], variants[j]}))
{
throw Exception(

1
tests/queries/0_stateless/03032_variant_bool_number_not_suspicious.reference Normal file
View File

@ -0,0 +1 @@
true

4
tests/queries/0_stateless/03032_variant_bool_number_not_suspicious.sql Normal file
View File

@ -0,0 +1,4 @@
set allow_experimental_variant_type=1;
set allow_suspicious_variant_types=0;
select 'true'::Bool::Variant(UInt32, Bool);