ydb/yql/essentials/sql/v1/complete/syntax/parser_call_stack.cpp, branch CLI_2.21.0Mirror of YDB github reposhttps://code.mastervirt.ru/ydb/atom?h=CLI_2.21.02025-05-16T12:32:43ZYQL-19747: Ignore type_id completions and table_key2025-05-16T12:32:43Zvityamanvityaman.dev@yandex.ru2025-05-16T12:20:12Zurn:sha1:31441fb3a8e27eb1c9f1125b55dd4a83d9310fc0
Fixes superfluous candidates (`OPTIONAL`, `CALLABLE`, ...) at `SELECT * FROM #`.
---
- Related to `YQL-19747`
- Related to https://github.com/ydb-platform/ydb/issues/9056
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1273
commit_hash:d34d81ea86b8b6b5d339174be20d14d0371c3de6
YQL-19747 Complete folder, table and cluster names2025-05-06T13:04:08Zvityamanvityaman.dev@yandex.ru2025-05-06T12:49:02Zurn:sha1:9c3fdca51d8ae892c5ad8f6ef92df73fafc09e28
---
- Related to `YQL-19747`
- On top of https://github.com/ytsaurus/ytsaurus/pull/1253
- Related to https://github.com/ydb-platform/ydb/issues/9056
- Related to https://github.com/vityaman/ydb/issues/14
- Related to https://github.com/vityaman/ydb/issues/35
- Related to https://github.com/vityaman/ydb/issues/40
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1257
commit_hash:0b842abb27184c88b8177beeea29fb1ea86b7a04
YQL-19747 Use TMaybe2025-04-28T10:52:54Zvityamanvityaman.dev@yandex.ru2025-04-28T10:29:56Zurn:sha1:3dfd8d9b20fe8902588dc35edb24423fbf165917
Just replaced `std::optional` usages with `TMaybe` to prevent this refactoring noise in future PRs.
---
- Related to `YQL-19747`
- Related to https://github.com/ydb-platform/ydb/issues/9056
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1247
commit_hash:dca8a6849e5ba9cb614d8350996f9423a1dc2373
YQL-19747 Complete read hints on PROCESS and REDUCE2025-04-10T14:35:51Zvityamanvityaman.dev@yandex.ru2025-04-10T14:08:20Zurn:sha1:473f2869597653cf26c8582095d1d9a805cb66fb
Forgot to support it.
- Related to https://github.com/ydb-platform/ydb/issues/9056
- Related to https://github.com/vityaman/ydb/issues/24
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1195
commit_hash:a2f5937dbca5712f3ecbfccdf66662ce99e70619
YQL-19747 Complete select and insert hints2025-04-09T15:12:05Zvityamanvityaman.dev@yandex.ru2025-04-09T14:57:35Zurn:sha1:394fee4aa8d48e58cf82c72bcb9dda0a5dd50190
- Related to https://github.com/ydb-platform/ydb/issues/9056
- Related to https://github.com/vityaman/ydb/issues/19
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1189
commit_hash:7f1cb1dcf0617aa2c94c3f2188fc9bd481380252
YQL-19747 Complete after PRAGMA and multi-token names2025-04-08T13:26:47Zvityamanvityaman.dev@yandex.ru2025-04-08T13:11:56Zurn:sha1:2f90258cf6f1625ba0c99f6f37b6c9f336590534
- [x] Complete after PRAGMA
- [x] Complete multi-token names correctly, for example, `yt.` returns only `DisableStrict`, not `yt.DisableStrict` and `DateTime::` returns `Split`, not `DateTime::Split`.
I tried to implement it using `CompletedToken` edition, but not all completion environments support candidates with various `contextLen` (`Replxx` does not). So I decided that completions should rewrite only the current token, not sequences. For example, on `DateTime::Spl` rewrite only `Spl`. It makes sense as multi-token names have some namespace separated by a punctuation, so used types only namespace and gets names inside of it.
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1181
commit_hash:9d8967ac43b9348f6dbb53837d92a9dcc9b51f48
YQL-19747 Complete type name as a function argument2025-04-07T12:00:33Zvityamanvityaman.dev@yandex.ru2025-04-07T11:33:48Zurn:sha1:c33aace83a03e01fbf260c3a28c614a58cbce812
As I understand, type name should not be completed at `SELECT |`, so I added a check that we are at `invoke_expr` context.
Currently composite type keywords are suggested at `SELECT |` and also are uppercased. I will fix it separately when this merged during
- https://github.com/users/vityaman/projects/5?pane=issue&itemId=105056723&issue=vityaman%7Cydb%7C8
---
- Related to https://github.com/ydb-platform/ydb/issues/9056
- Related to https://github.com/users/vityaman/projects/5/views/1?pane=issue&itemId=105056423&issue=vityaman%7Cydb%7C7
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1182
commit_hash:e87565867cf9fa82d9ac49a88d59b293d6686fe7
YQL-19747 Complete Function Names2025-03-28T19:36:16Zvityamanvityaman.dev@yandex.ru2025-03-28T19:01:21Zurn:sha1:b60cb8f5ce78ae5f63304a534278e124d31398b0
- Function names are suggested now
- Changed the module structure
- Checking ruleIndex independence on mode (ansi | default) via unit tests
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1163
commit_hash:1b1a27d2cff8db663c5c7e8efb57896476823315