Skip to content

Conversation

@vporyadke
Copy link
Collaborator

Changelog entry

...

Changelog category

  • Documentation (changelog entry is not required)

Description for reviewers

...

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 6, 2025
@github-actions
Copy link

github-actions bot commented Jun 6, 2025

🟢 2025-06-06 10:54:51 UTC The validation of the Pull Request description is successful.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@ydbot

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@vporyadke vporyadke marked this pull request as ready for review June 6, 2025 13:14
@vporyadke vporyadke requested a review from a team as a code owner June 6, 2025 13:14
@vporyadke vporyadke requested review from CyberROFL and ElenaAfina June 9, 2025 08:54
@ydbot

This comment was marked as outdated.

Co-authored-by: Ilnaz Nizametdinov <i.nizametdinov@gmail.com>
@ydbot

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@ydbot

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@ydb-platform ydb-platform deleted a comment from ydbot Oct 14, 2025
@ydb-platform ydb-platform deleted a comment from ydbot Oct 14, 2025
@ydb-platform ydb-platform deleted a comment from ydbot Oct 14, 2025
@ydb-platform ydb-platform deleted a comment from ydbot Oct 14, 2025
@ydb-platform ydb-platform deleted a comment from ydbot Oct 14, 2025
@ydb-platform ydb-platform deleted a comment from ydbot Oct 14, 2025
@ydbot

This comment was marked as outdated.

@CyberROFL CyberROFL mentioned this pull request Oct 16, 2025
8 tasks
This was linked to issues Oct 16, 2025
@CyberROFL CyberROFL assigned vporyadke and unassigned lopatinevgeny Oct 16, 2025
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions
Copy link

github-actions bot commented Oct 31, 2025

2025-10-31 16:56:09 UTC Pre-commit check linux-x86_64-relwithdebinfo for 9460d4c has started.
2025-10-31 16:56:29 UTC Artifacts will be uploaded here
2025-10-31 16:57:54 UTC ya make is running...
🟢 2025-10-31 16:58:04 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-10-31 16:58:09 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Oct 31, 2025

2025-10-31 16:56:16 UTC Pre-commit check linux-x86_64-release-asan for 9460d4c has started.
2025-10-31 16:56:34 UTC Artifacts will be uploaded here
2025-10-31 16:57:57 UTC ya make is running...
🟢 2025-10-31 16:58:06 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-10-31 16:58:11 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (4)

⚠️ en/devops/observability/monitoring.md: 54: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 7]

⚠️ ru/devops/observability/monitoring.md: 72: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 8]

⚠️ ru/yql/reference/syntax/insert_into.md: 65: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/yql/reference/syntax/update.md: 37: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

@CyberROFL CyberROFL requested review from a team and removed request for ElenaAfina November 1, 2025 08:24
@CyberROFL CyberROFL removed their assignment Nov 1, 2025
Copy link
Member

@CyberROFL CyberROFL left a comment

Choose a reason for hiding this comment

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

lgtm (шип не ставлю, чтобы случайно не закоммитилось раньше времени)


В данной статье описывается процесс выдачи [таблеткам](../concepts/glossary.md#tablet) [групп хранения](../concepts/glossary.md#storage-group) со стороны [Hive](../concepts/glossary.md#hive). При создании таблетки Hive назначает ей группы и в дальнейшем может их изменить:

* Если в группе заканчивается место, таблетки получают об этом сигнал от [прокси](../concepts/glossary.md#ds-proxy) и передают в Hive запрос выдать им другую группу.
Copy link
Member

Choose a reason for hiding this comment

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

Слово "прокси" запрещено к использованию. У нас так много просей, что нужно указывать конкретную, например "DS-прокси". И в глоссарии тоже нужно поправить

@@ -0,0 +1,47 @@
# Управление группами хранения в Hive

В данной статье описывается процесс выдачи [таблеткам](../concepts/glossary.md#tablet) [групп хранения](../concepts/glossary.md#storage-group) со стороны [Hive](../concepts/glossary.md#hive). При создании таблетки Hive назначает ей группы и в дальнейшем может их изменить:
Copy link
Member

Choose a reason for hiding this comment

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

На самом деле не хватает термина "таблетко-канал" или лучше "канал таблетки". Группы выдаются не таблетками, а каналам таблеток. Это бы написать.

Для безопасной смены группы Hive:

* Осуществляет блокировку записи в текущем [поколении](../concepts/glossary.md#tablet-generation) в системном канале таблетки.
* Запускает новое поколение таблетки с новой записью в [истории](general-schema.md#history).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Запускает новое поколение таблетки с новой записью в [истории](general-schema.md#history).
* Запускает новое поколение таблетки с новой записью в [истории каналов таблетки](general-schema.md#history).

## Единицы аллокации {#allocation-units}

Как и для [выбора узла](hive-booting.md#findbestnode) для запуска таблеток, для выбора групп в Hive предусмотрен учёт различных ресурсов. Однако, смена группы — более редкая операция, поэтому мгновенные значения потребления для неё бы не подошли. Необходимо ещё на этапе создания таблеток предсказать их потребление в долгосрочной перспективе. Для этого используются фиксированные значения: [SchemeShard](../concepts/glossary.md#scheme-shard) выставляет для каждого из каналов [единицу аллокации](../concepts/glossary.md#allocation-unit) — ожидаемое потребление ресурсов этим каналом, и Hive в дальнейшем использует эти значения. Единицами аллокации для различных таблеток можно управлять при помощи секции `channel_profile_config` в конфигурации кластера. Используются метрики:
Copy link
Member

Choose a reason for hiding this comment

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

  1. Надо добавить ссылку на "при помощи секции channel_profile_config в конфигурации кластера"
  2. Что значит "смена группы — более редкая операция"? не понятно из контекста
* $\mathrm{free}(g)$ — реальное свободное место в группе $g$;
* $\alpha, \beta$ — константы. По умолчанию $\alpha = 0.2, \beta = 1.1$. Значения можно настроить в [конфигурации Hive](../reference/configuration/hive.md#storage) как `space_usage_penalty` и `space_usage_penalty_threshold` соответственно.

## Единицы аллокации {#allocation-units}
Copy link
Member

Choose a reason for hiding this comment

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

Мне кажется не хватает более простых слов про единицы аллокации и примера. А то не понятно, если не знаешь

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

Labels

documentation Improvements or additions to documentation

6 participants