-
Notifications
You must be signed in to change notification settings - Fork 624
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.
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
Update CHANGELOG.md for 2025-04-06 #16816
base: main
Are you sure you want to change the base?
Conversation
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
CHANGELOG.md
Outdated
@@ -5,6 +5,7 @@ | |||
* 15186:Increased [the query text limit size](../dev/system-views#query-metrics) in system views from 4 KB to 10 KB. [#15186](https://github.com/ydb-platform/ydb/pull/15186) ([spuchin](https://github.com/spuchin)) | |||
* 15693:Added a health check configuration that administrators can customize: the number of node restarts, tablets, the time difference between database dynodes, | |||
and timeout (by default, the maximum response time from healthcheck). Documentation is under construction. [#15693](https://github.com/ydb-platform/ydb/pull/15693) ([Andrei Rykov](https://github.com/StekPerepolnen)) | |||
* 16792:The patchset makes `ReplicateScalars` work only with WideStream I/O type in DQ. [#16792](https://github.com/ydb-platform/ydb/pull/16792) ([Igor Munkin](https://github.com/igormunkin)) | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@igormunkin Привет! Подскажи, это изменение будет видно для конечного пользователя YDB?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@galnat, привет. Нет, это скорее внутреннее изменение.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В таком случае стоит выбирать категорию изменения "Not for changelog". У нас есть инструкция, как заполнять описание и правильно выбрать категорию - https://ydb.tech/docs/ru/contributor/suggest-change#create_pr_desc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@galhat, т.е. в случае изменений по каким-то внутренностям, связанным с YQL изменениям в Arcadia всегда стоит выбирать Not for changelog
?
CHANGELOG.md
Outdated
* 16423:Changed behavior — `SHOW CREATE TABLE` now fails on views instead of producing wrong output. [#16423](https://github.com/ydb-platform/ydb/pull/16423) ([Daniil Demin](https://github.com/jepett0)) | ||
* 16768:Fixed altering of max_active_partition property of the topic with YQL query [#16768](https://github.com/ydb-platform/ydb/pull/16768) ([Nikolay Shestakov](https://github.com/nshestakov)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nshestakov Коля, как тебе такой вариант:
Fixed a bug where topic auto-partitioning was not working when configuration parameter max_active_partition
were set via ALTER TOPIC statement.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
У Коли не было возражений
CHANGELOG.md
Outdated
* 16423:Changed behavior — `SHOW CREATE TABLE` now fails on views instead of producing wrong output. [#16423](https://github.com/ydb-platform/ydb/pull/16423) ([Daniil Demin](https://github.com/jepett0)) | ||
* 16768:Fixed altering of max_active_partition property of the topic with YQL query [#16768](https://github.com/ydb-platform/ydb/pull/16768) ([Nikolay Shestakov](https://github.com/nshestakov)) | ||
* 16764:Fixed redirects in describe handler closes #16763 [#16764](https://github.com/ydb-platform/ydb/pull/16764) ([Alexey Efimov](https://github.com/adameat)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adameat подскажи, это заметное для конечного пользователя изменение? какой именно редирект у него не работал?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adameat подскажи, это заметное для конечного пользователя изменение? какой именно редирект у него не работал?
да, заметное... не работал редирект с эндпоинта кластера (сторадж нод) на ноды базы, как следствие, получалось разное поведение (например не было видно часть системных таблиц)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Договорились изменить на "Fixed redirects from cluster endpoints (storage nodes) to database nodes, resolving inconsistent behavior where some system tables were not visible."
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
This PR updates the CHANGELOG.md file for 2025-04-06.