thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-21 15:12:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix

Browse Source
This commit is contained in:
Alexander Kuzmenkov 2021-05-27 21:39:06 +03:00
parent 1c6479a7d9
commit 9fc51aef68
3 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/AggregateFunctions/AggregateFunctionFactory.cpp
View File

@ -92,8 +92,10 @@ AggregateFunctionPtr AggregateFunctionFactory::get(
// Pure window functions are not real aggregate functions. Applying
// combinators doesn't make sense for them, they must handle the
// nullability themselves.
if (nested_function->asWindowFunction())
// nullability themselves. Another special case is functions from Nothing
// that are rewritten to AggregateFunctionNothing, in this case
// nested_function is nullptr.
if (nested_function && nested_function->asWindowFunction())
{
return nested_function;
}

2
tests/queries/0_stateless/01591_window_functions.reference
View File

@ -1058,6 +1058,8 @@ settings max_block_size = 3;
select lagInFrame(toNullable(1)) over ();
0
select lagInFrameOrNull(1) over (); -- { serverError 36 }
select intDiv(1, NULL) x, toTypeName(x), max(x) over ();
\N Nullable(Nothing) \N
-- case-insensitive SQL-standard synonyms for any and anyLast
select
number,

1
tests/queries/0_stateless/01591_window_functions.sql
View File

@ -379,6 +379,7 @@ settings max_block_size = 3;
-- careful with auto-application of Null combinator
select lagInFrame(toNullable(1)) over ();
select lagInFrameOrNull(1) over (); -- { serverError 36 }
select intDiv(1, NULL) x, toTypeName(x), max(x) over ();
-- case-insensitive SQL-standard synonyms for any and anyLast
select