diff options
| author | YDBot <[email protected]> | 2025-10-23 18:38:13 +0000 |
|---|---|---|
| committer | YDBot <[email protected]> | 2025-10-23 18:38:13 +0000 |
| commit | 564cf7cb2255a107b4f44c18b2a1844041f20b4d (patch) | |
| tree | 6a4a60e8bde612dcc3ac7fd93fffbd36351d4e35 /yql/essentials/sql/v1/sql_group_by.cpp | |
| parent | 89a17b25091c24744b7ebd0650b5b810457b1145 (diff) | |
| parent | a703d86902fd02bd8e373d959b2498c034657449 (diff) | |
Merge pull request #27203 from ydb-platform/merge-rightlib-251021-0051
Diffstat (limited to 'yql/essentials/sql/v1/sql_group_by.cpp')
| -rw-r--r-- | yql/essentials/sql/v1/sql_group_by.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/yql/essentials/sql/v1/sql_group_by.cpp b/yql/essentials/sql/v1/sql_group_by.cpp index deca5b6e534..cce460d9746 100644 --- a/yql/essentials/sql/v1/sql_group_by.cpp +++ b/yql/essentials/sql/v1/sql_group_by.cpp @@ -275,7 +275,7 @@ bool TGroupByClause::GroupingElement(const TRule_grouping_element& node, EGroupB break; } case TRule_grouping_element::ALT_NOT_SET: - Y_ABORT("You should change implementation according to grammar changes"); + Y_UNREACHABLE(); } return true; } @@ -435,8 +435,11 @@ bool TGroupByClause::AllowUnnamed(TPosition pos, EGroupByFeatures featureContext case EGroupByFeatures::GroupingSet: feature = "GROUPING SETS"; break; - default: + case EGroupByFeatures::Expression: + case EGroupByFeatures::Empty: + case EGroupByFeatures::End: YQL_ENSURE(false, "Unknown feature"); + break; } Ctx_.Error(pos) << "Unnamed expressions are not supported in " << feature << ". Please use '<expr> AS <name>'."; |