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

Fix short circuit execution of toFixedString function

Browse Source
This commit is contained in:
avogar 2022-08-25 16:15:22 +02:00
parent 13de3d0665
commit 1844873dbc
3 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Functions/toFixedString.h
View File

@ -43,7 +43,7 @@ public:
size_t getNumberOfArguments() const override { return 2; }
bool isInjective(const ColumnsWithTypeAndName &) const override { return true; }
bool isSuitableForShortCircuitArgumentsExecution(const DataTypesWithConstInfo & /*arguments*/) const override { return false; }
bool isSuitableForShortCircuitArgumentsExecution(const DataTypesWithConstInfo & /*arguments*/) const override { return true; }
DataTypePtr getReturnTypeImpl(const ColumnsWithTypeAndName & arguments) const override
{

2
tests/queries/0_stateless/02408_to_fixed_string_short_circuit.reference Normal file
View File

@ -0,0 +1,2 @@
\N
\N

1
tests/queries/0_stateless/02408_to_fixed_string_short_circuit.sql Normal file
View File

@ -0,0 +1 @@
select if(number < 0, toFixedString(materialize('123'), 2), NULL) from numbers(2);