Skip to content

Conversation

@vporyadke
Copy link
Collaborator

Changelog entry

update hive ui reference

Changelog category

  • Documentation (changelog entry is not required)

Additional information

...

@vporyadke vporyadke requested a review from a team as a code owner October 23, 2024 07:47
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 23, 2024
@github-actions
Copy link

github-actions bot commented Oct 23, 2024

2024-10-23 07:50:50 UTC Pre-commit check linux-x86_64-relwithdebinfo for 24b33cb has started.
2024-10-23 07:51:02 UTC Artifacts will be uploaded here
2024-10-23 07:53:37 UTC ya make is running...
🟢 2024-10-23 07:55:09 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2024-10-23 07:55:14 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Oct 23, 2024

2024-10-23 07:51:06 UTC Pre-commit check linux-x86_64-release-asan for 24b33cb has started.
2024-10-23 07:51:16 UTC Artifacts will be uploaded here
2024-10-23 07:53:45 UTC ya make is running...
🟢 2024-10-23 07:55:08 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2024-10-23 07:55:14 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

* **Wait Queue** - количество таблеток, которые не имеют возможности подняться
* **Totals** - суммарное потребление ресурсов таблетками (cnt, cpu, mem, net)
* **Variance** - среднеквадратическое отклонение потребления ресурсов (cnt, cpu, mem, net) между узлами
* **Triggers** - значения показателей, которые используются для принятия решения об автобалансировке. Каждый сопровождается цветовым индикатором: зелёный означает, что значение находится на приемлемом уровне; жёлтый - что близко к порогу; красный - что значение превысило порог и сейчас должна быть запущена автобалансировка.

This comment was marked as resolved.

Copy link
Member

Choose a reason for hiding this comment

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

ставлю пока в драфт, т.к. зависит от этого другого PR в статусе драфт

@vporyadke vporyadke requested review from adameat and ijon October 23, 2024 08:00
@blinkov blinkov marked this pull request as draft October 28, 2024 02:50
@vporyadke vporyadke mentioned this pull request Jan 31, 2025
8 tasks
@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 10:15:37 UTC Pre-commit check linux-x86_64-relwithdebinfo for 1bcf411 has started.
2025-02-12 10:15:41 UTC Artifacts will be uploaded here
2025-02-12 10:18:07 UTC ya make is running...
🟢 2025-02-12 10:18:30 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 10:18:36 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 10:18:26 UTC Pre-commit check linux-x86_64-release-asan for 1bcf411 has started.
2025-02-12 10:18:39 UTC Artifacts will be uploaded here
2025-02-12 10:21:05 UTC ya make is running...
🟢 2025-02-12 10:21:10 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 10:21:16 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@vporyadke vporyadke marked this pull request as ready for review February 12, 2025 11:07
@vporyadke vporyadke requested a review from CyberROFL February 12, 2025 11:08
@vporyadke vporyadke linked an issue Feb 12, 2025 that may be closed by this pull request
8 tasks
@github-actions
Copy link

github-actions bot commented Feb 21, 2025

2025-02-21 09:02:12 UTC Pre-commit check linux-x86_64-relwithdebinfo for 54d0741 has started.
2025-02-21 09:02:17 UTC Artifacts will be uploaded here
2025-02-21 09:04:35 UTC ya make is running...
🟢 2025-02-21 09:04:40 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-21 09:04:49 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 21, 2025

2025-02-21 09:02:12 UTC Pre-commit check linux-x86_64-release-asan for 54d0741 has started.
2025-02-21 09:02:16 UTC Artifacts will be uploaded here
2025-02-21 09:04:36 UTC ya make is running...
🟢 2025-02-21 09:04:42 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-21 09:04:47 UTC Build successful.

* **Starting** - количество таблеток в процессе подъема
* **Running** - количество работающих таблеток
* **Types** - распределение таблеток по их типу
* **Types** - распределение таблеток по их типу. С помощью нажатия на конкретный тип можно запретить запускать таблетки данного типа на конкретном узле. Нажатием на тип в верху таблицы можно установить такой запрет на все узлы по умолчанию.
Copy link
Contributor

Choose a reason for hiding this comment

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

"в верху таблицы" это в заголовке?


## Settings

На странице **Settings** представлен список параметров конфигурации Hive, которыми можно управлять через интерфейс. Для каждого параметра указано три значения: текущее значение, значение из [конфигурации YDB](../../maintenance/manual/config-overview.md) и значение по умолчанию. Параметры, значения которых выставлены через интерфейс, выделены жирным. Выставленные через интерфейс значения имеют приоритет над значениями из конфигурации YDB.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
На странице **Settings** представлен список параметров конфигурации Hive, которыми можно управлять через интерфейс. Для каждого параметра указано три значения: текущее значение, значение из [конфигурации YDB](../../maintenance/manual/config-overview.md) и значение по умолчанию. Параметры, значения которых выставлены через интерфейс, выделены жирным. Выставленные через интерфейс значения имеют приоритет над значениями из конфигурации YDB.
На странице **Settings** представлен список параметров конфигурации Hive, которыми можно управлять через интерфейс. Для каждого параметра указано три значения: текущее значение, значение из [конфигурации YDB](../../maintenance/manual/config-overview.md) и значение по умолчанию. Параметры, значения которых выставлены через интерфейс, выделены полужирным шрифтом. Выставленные через интерфейс значения имеют приоритет над значениями из конфигурации YDB.
--- | --- | ---
MaxTabletsScheduled | Целое число | Максимальное число таблеток, одновременно находящихся в процессе старта на одном узле
MaxBootBatchSize | Целое число | Максимальное число таблеток из очереди на запуск, обрабатываемых за раз
DrainInflight | Целое число | Число таблеток, одновременно перезапускающихся в процессе Drain одного узла
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
DrainInflight | Целое число | Число таблеток, одновременно перезапускающихся в процессе Drain одного узла
DrainInflight | Целое число | Число таблеток, одновременно перезапускающихся в процессе плавного перемещения всех таблеток с одного узла (drain)
MinCounterScatterToBalance | Вещественное число | Порог метрики Scatter для ресурса Counter
MinNodeUsageToBalance | Вещественное число | Потребление ресурсов на узле ниже данного значения приравнивается к данному значению
MaxNodeUsageToKick | Вещественное число | Порог потребления ресурсов на узле для запуска emergency-автобалансировки
NodeUsageRangeToKick | Вещественное число | Автобалансировка считается нецелесообразной, если разнице в уровне потребления ресурсов между узлами меньше этого значения
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
NodeUsageRangeToKick | Вещественное число | Автобалансировка считается нецелесообразной, если разнице в уровне потребления ресурсов между узлами меньше этого значения
NodeUsageRangeToKick | Вещественное число | Автобалансировка считается нецелесообразной, если разница в уровне потребления ресурсов между узлами меньше этого значения
NodeUsageRangeToKick | Вещественное число | Автобалансировка считается нецелесообразной, если разнице в уровне потребления ресурсов между узлами меньше этого значения
ResourceChangeReactionPeriod | Целое число секунд | Частота обновления аггрегированной статистики потребления ресурсов
TabletKickCooldownPeriod | Целое число секунд | Минимальный период времени между перемещениями одной таблетки
NodeSelectStrategy | Выбор из списка | Стратегия выбора узла для запуска таблетки. Возможные варианты: взвешенно-случайный выбор на основе потребления, выбор узла с минимальным потреблением, выбор случайного узла среди 7% узлов с наименьшим потреблением, выбор случайного узла
Copy link
Contributor

Choose a reason for hiding this comment

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

Если таблицу сделать в многострочном формате (https://diplodoc.com/docs/ru/syntax/tables/multiline), то можно будет "Возможные варианты:" оформить в виде списка, так легче парсить текст...

MaxMovementsOnEmergencyBalancer | Целое число | Аналогично MaxMovementsOnAutoBalancer, но для emergency-балансировки
ContinueEmergencyBalancer | вкл/выкл | Аналогично ContinueAutoBalancer, но для emergency-балансировки
CheckMoveExpediency | вкл/выкл | Производится ли проверка целесообразности перемещений таблеток
SpaceUsagePenaltyThreshold | Вещественное число | При перевозе канала из-за кончающегося места в группе, пессимизировать группы, в которых свободное место отличается во столько или менее раз от исходной группы
Copy link
Contributor

Choose a reason for hiding this comment

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

Тут коэффициент указывается, отличие в разы? Не очень понятно...

MinGroupUsageToBalance | Вещественное число | Потребление ресурсов группы хранения ниже данного значения приравнивается к данному значению
StorageBalancerInflight | Целое число | Число таблеток, одновременно перезапускающихся во время балансировки каналов
LessSystemTabletsMoves | вкл/выкл | Минимизация перемещения системных таблеток при автобалансировке
BalancerIgnoreTabletTypes | Список типов таблеток, разделённый точкой с запятой | Типы таблеток, на которые не распротраняется автобалансировка
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
BalancerIgnoreTabletTypes | Список типов таблеток, разделённый точкой с запятой | Типы таблеток, на которые не распротраняется автобалансировка
BalancerIgnoreTabletTypes | Список типов таблеток, разделённый точкой с запятой | Типы таблеток, на которые не распространяется автобалансировка
DefaultUnitThroughput | Целое число байт/секунду | Значение по умолчанию для потребления пропускной способности одним каналом
DefaultUnitSize | Целое число байт | Значение по умолчанию для потребления места на дисках одним каналом
StorageBalanceStrategy | Выбор из списка | Какой из параметров выше (IOPS, THROUGHPUT, SIZE) используется для распределения каналов таблеток по группам хранения. Вариант AUTO --- использование того, чьё потребление максимально.
StorageSafeMode | вкл/выкл | Проверяется ли превышение максимального потребления ресурсов групп хранения
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
StorageSafeMode | вкл/выкл | Проверяется ли превышение максимального потребления ресурсов групп хранения
StorageSafeMode | вкл/выкл | Проверка превышения максимального потребления ресурсов групп хранения
EmergencyBalancerInflight | Целое число | Аналогично BalancerInflight, но для emergency-балансировки
MaxMovementsOnEmergencyBalancer | Целое число | Аналогично MaxMovementsOnAutoBalancer, но для emergency-балансировки
ContinueEmergencyBalancer | вкл/выкл | Аналогично ContinueAutoBalancer, но для emergency-балансировки
CheckMoveExpediency | вкл/выкл | Производится ли проверка целесообразности перемещений таблеток
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
CheckMoveExpediency | вкл/выкл | Производится ли проверка целесообразности перемещений таблеток
CheckMoveExpediency | вкл/выкл | Проверка целесообразности перемещений таблеток
SpaceUsagePenalty | Вещественное число | Коэффициент штрафа, описанного выше
WarmUpBootWaitingPeriod | Целое число миллисекунд | Время ожидания старта всех известных узлов при старте базы
MaxWarmUpPeriod | Целое число секунд | Максимальное время ожидания старта узлов при старте базы
WarmUpEnabled | вкл/выкл | Дожидаться ли старта всех узлов при старте базы или запускать таблетки на первом подключившемся
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
WarmUpEnabled | вкл/выкл | Дожидаться ли старта всех узлов при старте базы или запускать таблетки на первом подключившемся
WarmUpEnabled | вкл/выкл | Ожидание старта всех узлов при старте базы или запуск таблетки на первом подключившемся
@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

❌ Documentation build

Revision build failed

Build logs

Errors (3)

❌ /ru/reference/embedded-ui/hive.md: 116: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

❌ /ru/reference/embedded-ui/hive.md: 117: MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- WEIGHTED_RANDOM — взвешенн..."]

❌ /ru/reference/embedded-ui/hive.md: 131: MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

@github-actions
Copy link

github-actions bot commented Feb 24, 2025

2025-02-24 11:02:00 UTC Pre-commit check linux-x86_64-relwithdebinfo for 8e0d4b1 has started.
2025-02-24 11:02:15 UTC Artifacts will be uploaded here
2025-02-24 11:04:39 UTC ya make is running...
🟢 2025-02-24 11:04:45 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-24 11:04:50 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 24, 2025

2025-02-24 11:02:15 UTC Pre-commit check linux-x86_64-release-asan for 8e0d4b1 has started.
2025-02-24 11:02:28 UTC Artifacts will be uploaded here
2025-02-24 11:04:44 UTC ya make is running...
🟢 2025-02-24 11:04:49 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-24 11:04:55 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 25, 2025

2025-02-25 08:56:40 UTC Pre-commit check linux-x86_64-relwithdebinfo for bbde880 has started.
2025-02-25 08:57:24 UTC Artifacts will be uploaded here
2025-02-25 09:00:13 UTC ya make is running...
🟢 2025-02-25 09:00:18 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-25 09:00:24 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Feb 25, 2025

2025-02-25 08:59:03 UTC Pre-commit check linux-x86_64-release-asan for bbde880 has started.
2025-02-25 08:59:19 UTC Artifacts will be uploaded here
2025-02-25 09:01:39 UTC ya make is running...
🟢 2025-02-25 09:01:45 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-25 09:01:51 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Mar 21, 2025

2025-03-21 10:08:04 UTC Pre-commit check linux-x86_64-release-asan for c9b0659 has started.
2025-03-21 10:08:20 UTC Artifacts will be uploaded here
2025-03-21 10:10:32 UTC ya make is running...
🟢 2025-03-21 10:10:38 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-21 10:10:45 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Mar 21, 2025

2025-03-21 10:08:19 UTC Pre-commit check linux-x86_64-relwithdebinfo for c9b0659 has started.
2025-03-21 10:08:35 UTC Artifacts will be uploaded here
2025-03-21 10:10:49 UTC ya make is running...
🟢 2025-03-21 10:10:55 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-21 10:11:02 UTC Build successful.

Copy link
Member

@fomichev3000 fomichev3000 left a comment

Choose a reason for hiding this comment

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

В целом почти нормально уже выглядит. Я написал много комментариев, но вроде бы они не очень сложно исправляются

#|
|| Название параметра в конфигурации | Название параметра в Hive web-viewer | Формат | Описание | Значение по умолчанию ||
|| `max_tablets_scheduled` | MaxTabletsScheduled | Целое число | Максимальное число таблеток, одновременно находящихся в процессе старта на одном узле. | 100 ||
|| `max_boot_batch_size` | MaxBootBatchSize | Целое число | Максимальное число таблеток из очереди на запуск, обрабатываемых за раз. | 1000 ||
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
|| `max_boot_batch_size` | MaxBootBatchSize | Целое число | Максимальное число таблеток из очереди на запуск, обрабатываемых за раз. | 1000 ||
|| `max_boot_batch_size` | MaxBootBatchSize | Целое число | Максимальное число таблеток из очереди запуска Hive, обрабатываемых за раз. | 1000 ||

Возможно надо писать не "обрабатываемых за раз", а "которые Hive пытается запустить параллельно". Если я прав. Но без слова Hive вообще не понятно что за очередь и где находится

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggestion - применил. Это именно про "за раз" (за один вызов обработчика в актор-системе, но этого-то никто не поймёт), параллельность может оказаться и выше

- `HIVE_NODE_SELECT_STRATEGY_RANDOM` — выбор случайного узла.

| `HIVE_NODE_SELECT_STRATEGY_RANDOM_MIN_7P` ||
|| `default_tablet_limit` || Вложенная секция | Ограничения на запуск таблеток различных типов на одном узле. Указывается в формате списка, где каждый элемент имеет поля `type` и `max_count`. | Пустая секция ||
Copy link
Member

Choose a reason for hiding this comment

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

Читателю не понятно, что с этим делать?
hive_config:
default_tablet_limit:
- type: Datashard
max_count: 5
- type: Columnshard
max_count: 3

Или я вообще не угадал?
Откуда брать типы таблеток?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Как ты предалагал ниже, добавил в пример в конце. Про типы таблеток написал перед примером.


| `HIVE_NODE_SELECT_STRATEGY_RANDOM_MIN_7P` ||
|| `default_tablet_limit` || Вложенная секция | Ограничения на запуск таблеток различных типов на одном узле. Указывается в формате списка, где каждый элемент имеет поля `type` и `max_count`. | Пустая секция ||
|| `default_tablet_preference` | — | Вложенная секция | Приоритеты по выбору датацентров для запуска таблеток различных типов. Для каждого типа таблетки можно указать несколько групп датацентров. Датацентры внутри одной группы будут иметь одинаковый приоритет, а более ранняя группа будет иметь приориет над последующими. Пример формата:
Copy link
Member

Choose a reason for hiding this comment

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

Также не понятно, где брать типы таблеток

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Про типы таблеток написал перед примером

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

В итоге про типы решили завести отдельный issue #16256, тут пока можно оставить укзаание на UI, после приведения типов в глоссарие в порядок - добавить ссылку туда

```
| Пустая секция ||
|| `system_category_id` | — | Целое число | При указании любого отличного от 0 числа, все координаторы и медиаторы по возможности запускаются в одном и том же датацентре. | 1 ||
Copy link
Member

Choose a reason for hiding this comment

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

Просто очень странное название опции. Почему не "group_system_tablets" со значениями 0 или 1. или true/false. Вот опция ниже скорее нормально называется
Но окей, я готов пока не переименовывать

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Странное, всё так. Ничего тут с этим не делал.

| Пустая секция ||
|| `system_category_id` | — | Целое число | При указании любого отличного от 0 числа, все координаторы и медиаторы по возможности запускаются в одном и том же датацентре. | 1 ||
|| `enable_fast_tablet_move` | — | true/false | Данная опция позволяет не дожидаться остановки предыдущего поколения таблетки при запуске нового. | true ||
Copy link
Member

Choose a reason for hiding this comment

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

Наверное стоит предложение написать зачем эта опция нужна. Ибо совсем не понятно.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Была нужна только для того, чтобы иметь возможность выключить, если вдруг что пойдёт не так, когда это только релизили. Из описания в итоге убрал. Завёл #16272

|| `storage_balancer_inflight` | StorageBalancerInflight | Целое число | Число таблеток, одновременно перезапускающихся во время балансировки каналов. | 1 ||
|# {wide-content}

## Опции отслеживания перезапусков {#restarts}
Copy link
Member

Choose a reason for hiding this comment

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

перезапусков чего? надо явно написать

возможно стоит сделать 2 раздела и 2 таблицы -- для таблеток и для нод

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Разбил на два подраздела

|| `tablet_restarts_period` | — | Целое число миллисекунд | Размер окна, на котором считается количество рестартов таблетки для пессимизации запуска проблемных таблеток. | 1000 ||
|| `tablet_restarts_max_count` | — | Целое число | Количество рестартов на окне `tablet_restarts_period`, при превышении которого применяется пессимизация. | 2 ||
|| `postopone_start_period` | — | Целое число миллисекунд | Периодичность попыток запуска проблемных таблеток. | 1000 ||
|| `node_restarts_for_penalty` | NodeRestartsForPenalty | Целое число | Количество рестартов на окне `node_restart_watch_period`, после которого узлы получают понижение приоритета. | 3 ||
Copy link
Member

Choose a reason for hiding this comment

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

А какой размер окна для нод? Такой же как для таблеток? Не понятно

Update: увидел. Тем правильнее на 2 раздела разделить

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Разбил на два подраздела

|| `tablet_restart_watch_period` | — | Целое число секунд | Размер окна, на котором собирается статистика о числе рестартов таблетки. | 3600 ||
|| `node_restart_watch_period` | — | Целое число секунд | Размер окна, на котором собирается статистика о числе рестартов узла. | 3600 ||
|| `tablet_restarts_period` | — | Целое число миллисекунд | Размер окна, на котором считается количество рестартов таблетки для пессимизации запуска проблемных таблеток. | 1000 ||
|| `tablet_restarts_max_count` | — | Целое число | Количество рестартов на окне `tablet_restarts_period`, при превышении которого применяется пессимизация. | 2 ||
Copy link
Member

Choose a reason for hiding this comment

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

Я прочитал и не понял логику работы на текущих дефолтных настройках. Таблетка пессимизируется, когда 2 раза рестартовала на окне в 1000мс? Или на окне 3600 секунд?
А значит и пользователь скорее не поймет

Copy link
Member

Choose a reason for hiding this comment

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

я прочитал как «на окне в 1000мс»

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Да что там пользователь, тут и я могу перепутать :/

Дописал к окну 3600с пояснение, что оно для другого

Этот период используется только для статистики, передаваемой в HealthCheck. |

|| `node_restarts_for_penalty` | NodeRestartsForPenalty | Целое число | Количество рестартов на окне `node_restart_watch_period`, после которого узлы получают понижение приоритета. | 3 ||
|# {wide-content}

## Прочее {#misc}
Copy link
Member

Choose a reason for hiding this comment

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

Возможно я бы вставил сюда плашку "Эта таблица содержит продвинутые настройки, которые в большинстве случаев не требуют изменения."
Ибо не понятны половина+ опций...

Copy link
Member

Choose a reason for hiding this comment

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

вообще её можно было б прям начало страницы перенести)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Вставил эту плашку.


## Reassign Groups {#reassign_groups}

При нажатии на кнопку **Reassign Groups** появится окно с параметрами для балансировки:
Copy link
Member

Choose a reason for hiding this comment

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

Давай опишем зачем вообще надо нажимать эту кнопку. И предупреждение сделаем

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Дописал про сценарий добавления групп

@github-actions
Copy link

github-actions bot commented Mar 24, 2025

2025-03-24 18:07:08 UTC Pre-commit check linux-x86_64-relwithdebinfo for 6cb6c43 has started.
2025-03-24 18:07:14 UTC Artifacts will be uploaded here
2025-03-24 18:09:29 UTC ya make is running...
🟢 2025-03-24 18:09:35 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-24 18:09:42 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Mar 24, 2025

2025-03-24 18:09:53 UTC Pre-commit check linux-x86_64-release-asan for 6cb6c43 has started.
2025-03-24 18:10:09 UTC Artifacts will be uploaded here
2025-03-24 18:12:20 UTC ya make is running...
🟢 2025-03-24 18:12:26 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-24 18:12:34 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Mar 25, 2025

2025-03-25 08:15:15 UTC Pre-commit check linux-x86_64-release-asan for d40ac66 has started.
2025-03-25 08:15:31 UTC Artifacts will be uploaded here
2025-03-25 08:17:51 UTC ya make is running...
🟢 2025-03-25 08:17:58 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-25 08:18:06 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions
Copy link

github-actions bot commented Mar 25, 2025

2025-03-25 08:17:56 UTC Pre-commit check linux-x86_64-relwithdebinfo for d40ac66 has started.
2025-03-25 08:18:14 UTC Artifacts will be uploaded here
2025-03-25 08:20:36 UTC ya make is running...
🟢 2025-03-25 08:20:43 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-25 08:20:49 UTC Build successful.

@vporyadke vporyadke requested review from anton-bobkov and blinkov and removed request for blinkov March 25, 2025 08:34
@blinkov blinkov enabled auto-merge (squash) March 26, 2025 00:53
Copy link
Member

@fomichev3000 fomichev3000 left a comment

Choose a reason for hiding this comment

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

Я посмотрел последний кормит, он не адресует все мои комментарии. Например, список таблеток я так и не вижу.
Надо последовательно адресовать список всех комментариев!

@vporyadke
Copy link
Collaborator Author

Я посмотрел последний кормит, он не адресует все мои комментарии. Например, список таблеток я так и не вижу. Надо последовательно адресовать список всех комментариев!

Я думал, что так и сделал. В частности про таблетки я написал перед примером

В подсекциях default_tablet_limit и default_tablet_preference нужно указывать типы таблеток. Точные названия типов
таблеток можно посмотреть, например, на вкладке Tablets в YDB UI.

Это наверное не идеальный ответ. Правильный ответ - список всех типов таблеток живёт в tablet_types.proto, кстати там же указаны числовые айдишники и можно при желании использовать их. Но отправлять читателя документации смотреть в .proto я всё-таки не хочу. Ты хотел увидеть прямо список из всех типов таблеток в документации? В теории его можно бы было выделить в глоссарии, но там всякие тонкие моменты, кого в него включать, кого нет: nbs'ные и rtmr'ные таблетки наверное не нужно, а бывает ли у внешних польователей PersQueueReadBalancer или ReplicationController? Можно ли уже упоминать блобовницу и графшард?

@github-actions
Copy link

github-actions bot commented Mar 26, 2025

2025-03-26 11:35:40 UTC Pre-commit check linux-x86_64-release-asan for a76cd2c has started.
2025-03-26 11:35:57 UTC Artifacts will be uploaded here
2025-03-26 11:38:19 UTC ya make is running...
🟢 2025-03-26 11:38:26 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-26 11:38:33 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Mar 26, 2025

2025-03-26 11:37:47 UTC Pre-commit check linux-x86_64-relwithdebinfo for a76cd2c has started.
2025-03-26 11:38:02 UTC Artifacts will be uploaded here
2025-03-26 11:40:20 UTC ya make is running...
🟢 2025-03-26 11:40:27 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-03-26 11:40:33 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@vporyadke vporyadke requested review from anton-bobkov and blinkov and removed request for anton-bobkov and blinkov March 26, 2025 17:19
@blinkov blinkov merged commit a94e17d into ydb-platform:main Mar 27, 2025
13 checks passed
lberserq pushed a commit to lberserq/ydb that referenced this pull request Mar 28, 2025
Co-authored-by: Ivan Blinkov <ivan@ydb.tech> Co-authored-by: Ilnaz Nizametdinov <i.nizametdinov@gmail.com> Co-authored-by: anton-bobkov <anton-bobkov@ydb.tech>
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

5 participants