-
Notifications
You must be signed in to change notification settings - Fork 606
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
Add few auto partitioning fields to describe and SDK. Add important consumer to CLI #6118
Add few auto partitioning fields to describe and SDK. Add important consumer to CLI #6118
Conversation
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
std::sort(childIds.begin(), childIds.end()); | ||
UNIT_ASSERT_EQUAL(childIds[0], 1); | ||
UNIT_ASSERT_EQUAL(childIds[1], 2); | ||
} |
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.
add a break here
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
@@ -1087,6 +1087,11 @@ message DescribeTopicResult { | |||
|
|||
// Partition location, filled only when include_location in request is true. | |||
PartitionLocation partition_location = 6; | |||
|
|||
// Inclusive left border. Emptiness means -inf. | |||
optional bytes from_bound = 7; |
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.
А что там внутри bytes?
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.
Строка. Но там могут быть нечитаемые символы, поэтому байты. Границы не человекочитаемы
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.
Ну вот непонятно как этим пользоваться тогда?
Какой формат, если это бинарные данные?
Что тут будет SDK/Describe возвращать?
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.
Дескрайб в SDK возвращает TString. Вот пример https://github.com/ydb-platform/ydb/pull/6118/files#diff-c7baefa581e4001940d02d8510c369295cc6c020f946c62bc1f1fdccdbd37e61R824
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.
Как пользоваться - лексикографически сравнивать. Можно, например, определить, лежит ли твой ключ в границах партиции просто сравнением
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
Changelog entry
Changelog category
Additional information
...