Skip to content

(docs) append create table syntax #30484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
orange13 merged 4 commits into from
Dec 15, 2025
Merged

(docs) append create table syntax #30484

orange13 merged 4 commits into from
Dec 15, 2025

Conversation

@orange13
Copy link
Contributor

@orange13 orange13 commented Dec 10, 2025

Changelog entry

...

Changelog category

  • Documentation (changelog entry is not required)

Description for reviewers

...

@orange13 orange13 requested a review from a team as a code owner December 10, 2025 14:42
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 10, 2025
@ydbot
Copy link
Collaborator

ydbot commented Dec 10, 2025

Run Extra Tests

Run additional tests for this PR. You can customize:

  • Test Size: small, medium, large (default: all)
  • Test Targets: any directory path (default: ydb/)
  • Sanitizers: ASAN, MSAN, TSAN
  • Coredumps: enable for debugging (default: off)
  • Additional args: custom ya make arguments

▶ Run tests

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

🟢 2025-12-10 14:43:18 UTC The validation of the Pull Request description is successful.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025 •
edited
Loading

⚪ 2025-12-10 14:44:50 UTC Pre-commit check linux-x86_64-relwithdebinfo for 75e75ed has started.
⚪ 2025-12-10 14:44:55 UTC Artifacts will be uploaded here
⚪ 2025-12-10 14:47:04 UTC ya make is running...
🟢 2025-12-10 14:47:10 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-10 14:47:16 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025 •
edited
Loading

⚪ 2025-12-10 14:44:53 UTC Pre-commit check linux-x86_64-release-asan for 75e75ed has started.
⚪ 2025-12-10 14:44:57 UTC Artifacts will be uploaded here
⚪ 2025-12-10 14:47:01 UTC ya make is running...
🟢 2025-12-10 14:47:07 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-10 14:47:13 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (4)

⚠️ Incorrect syntax for notes, file ru/devops/observability/alerting.md

⚠️ Skip error: SkippedEvalError: Cannot apply an unsupported filter: ansible_config_file | dirname

⚠️ Variable ansible_config_file | dirname not found in ru/devops/deployment-options/ansible/initial-deployment.md

⚠️ ru/reference/ydb-cli/topic-consumer-add.md: 35: MD001 / heading-increment Heading levels should only increment by one level at a time [Expected: h3; Actual: h4]

dorooleg
dorooleg requested changes Dec 10, 2025
View reviewed changes
@github-actions
Copy link

github-actions bot commented Dec 10, 2025 •
edited
Loading

⚪ 2025-12-10 15:09:36 UTC Pre-commit check linux-x86_64-relwithdebinfo for bb4269d has started.
⚪ 2025-12-10 15:11:03 UTC Artifacts will be uploaded here
⚪ 2025-12-10 15:13:05 UTC ya make is running...
🟢 2025-12-10 15:13:11 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-10 15:13:17 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025 •
edited
Loading

⚪ 2025-12-10 15:10:33 UTC Pre-commit check linux-x86_64-release-asan for bb4269d has started.
⚪ 2025-12-10 15:10:51 UTC Artifacts will be uploaded here
⚪ 2025-12-10 15:14:06 UTC ya make is running...
🟢 2025-12-10 15:14:12 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-10 15:14:19 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (4)

⚠️ Incorrect syntax for notes, file ru/devops/observability/alerting.md

⚠️ Skip error: SkippedEvalError: Cannot apply an unsupported filter: ansible_config_file | dirname

⚠️ Variable ansible_config_file | dirname not found in ru/devops/deployment-options/ansible/initial-deployment.md

⚠️ ru/reference/ydb-cli/topic-consumer-add.md: 35: MD001 / heading-increment Heading levels should only increment by one level at a time [Expected: h3; Actual: h4]

@azevaykin azevaykin requested a review from ztlpn December 10, 2025 15:11
ztlpn
ztlpn reviewed Dec 10, 2025
View reviewed changes
Copy link
Collaborator

@ztlpn ztlpn left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Может быть не стоит сейчас публиковать этот синтаксис, так как он

  1. Не добавляет сейчас никаких новых возможностей пользователю и
  2. Не слишком консистентен с колоночными таблицами (PARTITION BY без указания способа партиционирования, располагается в списке колонок, а не в конце CREATE TABLE).

Мы в следующем полугодии будем добавлять новые возможности и партиционирование строковых таблиц, наверно там будет другой синтаксис.

azevaykin
azevaykin reviewed Dec 11, 2025
View reviewed changes
@github-actions
Copy link

github-actions bot commented Dec 11, 2025 •
edited
Loading

⚪ 2025-12-11 11:27:02 UTC Pre-commit check linux-x86_64-relwithdebinfo for c412ba2 has started.
⚪ 2025-12-11 11:27:19 UTC Artifacts will be uploaded here
⚪ 2025-12-11 11:29:25 UTC ya make is running...
🟢 2025-12-11 11:29:31 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-11 11:29:36 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 11, 2025

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (4)

⚠️ Incorrect syntax for notes, file ru/devops/observability/alerting.md

⚠️ Skip error: SkippedEvalError: Cannot apply an unsupported filter: ansible_config_file | dirname

⚠️ Variable ansible_config_file | dirname not found in ru/devops/deployment-options/ansible/initial-deployment.md

⚠️ ru/reference/ydb-cli/topic-consumer-add.md: 35: MD001 / heading-increment Heading levels should only increment by one level at a time [Expected: h3; Actual: h4]

@github-actions
Copy link

github-actions bot commented Dec 11, 2025 •
edited
Loading

⚪ 2025-12-11 11:27:33 UTC Pre-commit check linux-x86_64-release-asan for c412ba2 has started.
⚪ 2025-12-11 11:27:50 UTC Artifacts will be uploaded here
⚪ 2025-12-11 11:29:57 UTC ya make is running...
🟢 2025-12-11 11:30:03 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-11 11:30:08 UTC Build successful.

dorooleg
dorooleg previously approved these changes Dec 11, 2025
View reviewed changes
fomichev3000
fomichev3000 previously approved these changes Dec 13, 2025
View reviewed changes
@orange13 @fomichev3000
Update ydb/docs/ru/core/yql/reference/syntax/create_table/index.md
39a84e7 
Co-authored-by: Andrey Fomichev <fomichev@ydb.tech>
@orange13 orange13 dismissed stale reviews from fomichev3000 and dorooleg via 39a84e7 December 15, 2025 12:48
@github-actions
Copy link

github-actions bot commented Dec 15, 2025 •
edited
Loading

⚪ 2025-12-15 12:52:00 UTC Pre-commit check linux-x86_64-relwithdebinfo for 81d17e6 has started.
⚪ 2025-12-15 12:52:30 UTC Artifacts will be uploaded here
⚪ 2025-12-15 12:54:24 UTC ya make is running...
🟢 2025-12-15 12:54:31 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-15 12:54:37 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 15, 2025 •
edited
Loading

⚪ 2025-12-15 12:52:22 UTC Pre-commit check linux-x86_64-release-asan for 81d17e6 has started.
⚪ 2025-12-15 12:52:40 UTC Artifacts will be uploaded here
⚪ 2025-12-15 12:54:43 UTC ya make is running...
🟢 2025-12-15 12:54:49 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-12-15 12:54:54 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 15, 2025

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (4)

⚠️ Incorrect syntax for notes, file ru/devops/observability/alerting.md

⚠️ Skip error: SkippedEvalError: Cannot apply an unsupported filter: ansible_config_file | dirname

⚠️ Variable ansible_config_file | dirname not found in ru/devops/deployment-options/ansible/initial-deployment.md

⚠️ ru/reference/ydb-cli/topic-consumer-add.md: 35: MD001 / heading-increment Heading levels should only increment by one level at a time [Expected: h3; Actual: h4]

@orange13 orange13 enabled auto-merge (squash) December 15, 2025 12:58
a-s-maslov
a-s-maslov approved these changes Dec 15, 2025
View reviewed changes
Copy link
Collaborator

@a-s-maslov a-s-maslov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@orange13 orange13 merged commit a852003 into main Dec 15, 2025
10 checks passed
@ydbot
Copy link
Collaborator

ydbot commented Dec 15, 2025

Backport

To backport this PR, click the button next to the target branch and then click "Run workflow" in the Run Actions UI.

Branch Run
stable-25-2, stable-25-2-1, stable-25-3, stable-25-3-1 ▶ Backport
stable-25-3, stable-25-3-1 ▶ Backport
stable-25-3 ▶ Backport

▶ Backport manual

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ztlpn ztlpn ztlpn left review comments

@azevaykin azevaykin azevaykin left review comments

@a-s-maslov a-s-maslov a-s-maslov approved these changes

@fomichev3000 fomichev3000 fomichev3000 left review comments

@dorooleg dorooleg dorooleg left review comments

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@orange13 @ydbot @ztlpn @fomichev3000 @dorooleg @azevaykin @a-s-maslov