Skip to content

Conversation

vitalyisaev2
Copy link
Member

Changelog entry

Changelog category

  • Documentation (changelog entry is not required)

Description for reviewers

...

@vitalyisaev2 vitalyisaev2 requested a review from a team as a code owner October 16, 2025 12:20
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 16, 2025
Copy link

🟢 2025-10-16 12:23:17 UTC The validation of the Pull Request description is successful.

This comment was marked as outdated.

This comment was marked as outdated.

@vitalyisaev2 vitalyisaev2 marked this pull request as draft October 16, 2025 12:33
@vitalyisaev2 vitalyisaev2 marked this pull request as ready for review October 16, 2025 12:33

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

Copy link

github-actions bot commented Oct 16, 2025

2025-10-16 16:08:32 UTC Pre-commit check linux-x86_64-release-asan for 4c600db has started.
2025-10-16 16:08:54 UTC Artifacts will be uploaded here
2025-10-16 16:12:10 UTC ya make is running...
🟢 2025-10-16 16:12:16 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-10-16 16:12:19 UTC Build successful.

Copy link

github-actions bot commented Oct 16, 2025

2025-10-16 16:09:17 UTC Pre-commit check linux-x86_64-relwithdebinfo for 4c600db has started.
2025-10-16 16:09:21 UTC Artifacts will be uploaded here
2025-10-16 16:11:58 UTC ya make is running...
🟢 2025-10-16 16:12:04 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-10-16 16:12:07 UTC Build successful.

| Параметр | Назначение | Значение по умолчанию |
|----------|------------|-----------------------|
| `connector_server` | Обязательная секция. Содержит настройки основного GPRC-сервера, выполняющего доступ к данным. ||
| `connector_server.endpoint.host` | Хостнейм или IP-адрес, на котором запускается слушающий сокет сервиса. ||
Copy link
Collaborator

Choose a reason for hiding this comment

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

Хостнейм - это не по-русски

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]

|----------|------------|-----------------------|
| `connector_server` | Обязательная секция. Содержит настройки основного GPRC-сервера, выполняющего доступ к данным. ||
| `connector_server.endpoint.host` | Хостнейм или IP-адрес, на котором запускается слушающий сокет сервиса. ||
| `connector_server.endpoint.port` | Номер порта, на котором запускается слушающий сокет сервиса. ||
Copy link
Collaborator

Choose a reason for hiding this comment

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

слово "сокет" я бы удалил из документации. это ненужная деталь, которая усложняет понимание.
приложение слушает на TCP порту или приложение принимает входящие соединения через порт такой-то

| Параметр | Назначение | Значение по умолчанию |
|----------|------------|-----------------------|
| `connector_server` | Обязательная секция. Содержит настройки основного GPRC-сервера, выполняющего доступ к данным. ||
| `connector_server.endpoint.host` | Хостнейм или IP-адрес, на котором запускается слушающий сокет сервиса. ||
Copy link
Collaborator

Choose a reason for hiding this comment

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

тут можно предложить дефолт 127.0.0.1 ?

| `metrics_server` | Опциональная секция. Содержит настройки вспомогательного HTTP-сервера, выполняющего доступ к метрикам. | Сервер отключён. |
| `metrics_server.endpoint.host` | Хостнейм или IP-адрес, на котором запускается слушающий сокет сервиса. ||
| `metrics_server.endpoint.port` | Номер порта, на котором запускается слушающий сокет сервиса. ||
| `paging` | Опциональная секция. Содержит настройки алгоритма разбиения извлекаемого из источника потока данных на Arrow-блоки. На каждый запрос в коннекторе создаётся очередь из заранее подготовленных к отправке на сторону {{ ydb-short-name }} блоков данных. Аллокации Arrow-блоков формируют наиболее существенный вклад в потребление оперативной памяти процессом `fq-connector-go`. Минимальный объём памяти, необходимый коннектору для работы, можно приблизительно оценить по формуле $Mem = 2 \cdot Requests \cdot BPP \cdot PQC$, где $Requests$ — количество одновременно выполняемых запросов, $BPP$ — параметр `paging.bytes_per_page`, а $PQC$ — параметр `paging.prefetch_queue_capacity`. ||
Copy link
Collaborator

Choose a reason for hiding this comment

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

в формуле я бы убрал алиасы BPP и PQC
mem = requests * paging.bytes_per_page * paging.prefetch_queue_capacity

читается хорошо

| `datasources.clickhouse.pushdown` | Опциональная секция. Содержит различные настройки для пушдауна предикатов. ||
| `datasources.clickhouse.pushdown.enable_timestamp_pushdown` | Включает пушдаун фильтров для столбцов типа YQL `Timestamp`. Допустимые значения: `true`, `false`. | `false` |
| `datasources.mysql` | Опциональная секция. Содержит настройки, специфичные для источника данных MySQL. ||
| `datasources.mysql.result_chan_capacity` | Ёмкость буфера для извлечённых строк из сетевого соединения с базой. | `512` |
Copy link
Collaborator

Choose a reason for hiding this comment

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

в каких единицах? строки\байты?

| `datasources.greenplum` | Опциональная секция. Содержит настройки, специфичные для источника данных Greenplum. ||
| `datasources.greenplum.open_connection_timeout` | Таймаут открытия соединения с Greenplum. Варианты допустимых значений: `100ms`, `5s`, `3h`. | `5s` |
| `datasources.greenplum.pushdown` | Опциональная секция. Содержит различные настройки для пушдауна предикатов. ||
| `datasources.greenplum.pushdown.enable_timestamp_pushdown` | Включает пушадаун фильтров для столбцов типа YQL Timestamp. Допустимые значения: `true`, `false`. | `false` |
Copy link
Collaborator

Choose a reason for hiding this comment

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

опечатка в "пушадаун". и у тебя тут используется еще слово "проталкивание". выбери одно слово

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.

Начну смотреть, когда будут закрыты открытые комментарии и стоять шип от @uzhastik

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants