diff options
| author | uzhas <uzhas@ydb.tech> | 2023-02-10 16:20:16 +0300 |
|---|---|---|
| committer | uzhas <uzhas@ydb.tech> | 2023-02-10 16:20:16 +0300 |
| commit | 45f5cd6f4734b7204a9186f9b476e30dbfa839ad (patch) | |
| tree | a66ed68694f3b6c454dfb46b7c5e7e3fc71b3c5f | |
| parent | 6f49b8de56ec7992e8d93b5b4c0807cb28caa549 (diff) | |
| download | ydb-45f5cd6f4734b7204a9186f9b476e30dbfa839ad.tar.gz | |
less specific error message
improve error message
| -rw-r--r-- | ydb/library/yql/sql/v1/builtin.cpp | 4 | ||||
| -rw-r--r-- | ydb/library/yql/sql/v1/sql_ut.cpp | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/ydb/library/yql/sql/v1/builtin.cpp b/ydb/library/yql/sql/v1/builtin.cpp index eb5f0e39ecd..1626b9d302e 100644 --- a/ydb/library/yql/sql/v1/builtin.cpp +++ b/ydb/library/yql/sql/v1/builtin.cpp @@ -480,7 +480,7 @@ public: if (src->HasAggregations()) { ctx.Warning(Pos, TIssuesIds::YQL_EMPTY_TABLENAME_RESULT) << "TableName() will produce empty result when used with aggregation.\n" - "Please consult https://yql.yandex-team.ru/docs/yt/builtins/basic/#tablepath for possible workaround"; + "Please consult documentation for possible workaround"; } Args.push_back(Y("TablePath", Y("DependsOn", "row"))); @@ -496,7 +496,7 @@ public: Args[1] = BuildQuotedAtom(Args[1]->GetPos(), *literal); } else { if (Service.empty()) { - ctx.Error(GetPos()) << GetOpName() << " requires either one of \"yt\"/\"kikimr\"/\"rtmr\" as second argument or current cluster name"; + ctx.Error(GetPos()) << GetOpName() << " requires either service name as second argument or current cluster name"; return false; } diff --git a/ydb/library/yql/sql/v1/sql_ut.cpp b/ydb/library/yql/sql/v1/sql_ut.cpp index d7d48d74822..ce96a14d2d5 100644 --- a/ydb/library/yql/sql/v1/sql_ut.cpp +++ b/ydb/library/yql/sql/v1/sql_ut.cpp @@ -3195,7 +3195,7 @@ select FormatType($f()); auto res = SqlToYql("select TableName() from plato.Input;"); UNIT_ASSERT(!res.Root); - UNIT_ASSERT_NO_DIFF(Err2Str(res), "<main>:1:8: Error: TableName requires either one of \"yt\"/\"kikimr\"/\"rtmr\" as second argument or current cluster name\n"); + UNIT_ASSERT_NO_DIFF(Err2Str(res), "<main>:1:8: Error: TableName requires either service name as second argument or current cluster name\n"); res = SqlToYql("use plato;\n" "select TableName() from Input1 as a join Input2 as b using(key);"); @@ -3206,7 +3206,7 @@ select FormatType($f()); "select SOME(TableName()), key from Input group by key;"); UNIT_ASSERT(res.Root); UNIT_ASSERT_NO_DIFF(Err2Str(res), "<main>:2:13: Warning: TableName() will produce empty result when used with aggregation.\n" - "Please consult https://yql.yandex-team.ru/docs/yt/builtins/basic/#tablepath for possible workaround, code: 4525\n"); + "Please consult documentation for possible workaround, code: 4525\n"); } Y_UNIT_TEST(WarnOnDistincWithHavingWithoutAggregations) { |