thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-26 01:22:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary emtpy argument check for nth_value as code review suggested

Browse Source
This commit is contained in:
ryzuo 2021-07-26 13:31:45 +08:00
parent 4d36d54c81
commit 0e742487f1
1 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/Processors/Transforms/WindowTransform.cpp
View File

@ -9,6 +9,7 @@
#include <Interpreters/ExpressionActions.h> #include <Interpreters/ExpressionActions.h>
#include <Interpreters/convertFieldToType.h> #include <Interpreters/convertFieldToType.h>
namespace DB namespace DB
{ {
@ -1576,12 +1577,6 @@ struct WindowFunctionNthValue final : public WindowFunction
"Function {} cannot be parameterized", name_); "Function {} cannot be parameterized", name_);
} }
if (argument_types.empty())
{
throw Exception(ErrorCodes::BAD_ARGUMENTS,
"Function {} takes at least one argument", name_);
}
if (argument_types.size() != 2) if (argument_types.size() != 2)
{ {
throw Exception(ErrorCodes::BAD_ARGUMENTS, throw Exception(ErrorCodes::BAD_ARGUMENTS,