-
Notifications
You must be signed in to change notification settings - Fork 735
Create a sequence for prefixed vector indexes #26046
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
Create a sequence for prefixed vector indexes #26046
Conversation
|
🟢 |
f5c6c93 to
0724ecb
Compare
|
⚪
🟢
*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 |
| LockBuild = 47, | ||
| Applying = 50, | ||
| Unlocking = 60, | ||
| AlterSequence = 61, |
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.
Достоверно не проверял... Но есть ощущение, что построение зависнет, т.к. нигде в case-ах, где проверяются состояния, нет default: Y_ENSURE(false), везде просто проход мимо. Правда, если бы и было, то катастрофы бы тоже не случилось, т.к. сейчас вся обработка построения индекса и загрузки начального состояния построения индекса обёрнута в try-catch и если ловится исключение, то операция построения индекса переходит в состояние Broken.
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.
Нужно обеспечить, чтобы описанная мной ситуация была безопасной: флагом, обработкой нового состояния в старых версиях и т.п.
CyberROFL
left a comment
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.
Шипаю в долг: #26046 (comment)
Ок, спасибо. Но я уже пробую тест на этот кейс написать) |
Changelog category
Description for reviewers
Extracted from #25505, schemeshard part - sequence creation and value reset.