-
Notifications
You must be signed in to change notification settings - Fork 735
Add paragraph about Implicit Transaction Control (NoTx mode) #21324
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
Conversation
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
Hey @lopatinevgeny, it has been 25 business-hours since the author's last update, could you please review? |
|
Hey @lopatinevgeny, it has been 49 business-hours since the author's last update, could you please review? |
|
Hey @lopatinevgeny, it has been 73 business-hours since the author's last update, could you please review? |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
🔄 New commits pushed — @lopatinevgeny please take a look. |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
🔄 New commits pushed — @lopatinevgeny please take a look. |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
🔄 New commits pushed — @lopatinevgeny please take a look. |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
Hey @lopatinevgeny, it has been 121 business-hours since the author's last update, could you please review? |
|
Hey @lopatinevgeny, it has been 151 business-hours since the author's last update, could you please review? |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
🔄 New commits pushed — @lopatinevgeny please take a look. |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
🔄 New commits pushed — @lopatinevgeny please take a look. |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
Hey @lopatinevgeny, it has been 31 business-hours since the author's last update, could you please review? |
|
Hey @lopatinevgeny, it has been 55 business-hours since the author's last update, could you please review? |
|
Hey @lopatinevgeny, it has been 85 business-hours since the author's last update, could you please review? |
|
🔄 New commits pushed — @lopatinevgeny please take a look. |
✅ Documentation buildRevision built successfully Build logsWarnings (4)
|
|
⚪ Test history | Ya make output | Test bloat
🟢 |
|
⚪ Test history | Ya make output | Test bloat
🟢 |
fomichev3000
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.
В целом можно закоммитить в таком виде, чтобы не держать. Но я бы хотел, чтобы были поправлены замечания. Многие из замечаний исправляются супер быстро
|
|
||
| В этом режиме {{ ydb-short-name }} на основе запроса определяет, выполнить его вне транзакции или обернуть в транзакцию с режимом *Serializable*. Режим неявной транзакции является универсальным для выполнения запроса, так как поддерживает инструкции любого вида с определённым поведением, описанным ниже. | ||
|
|
||
| #### Поведение для разных видов инструкций |
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.
Я бы убрал заголовок. Вот просто совсем без него.
| - **Инструкции пакетного изменения** | ||
| Инструкции пакетного изменения (такие как [BATCH UPDATE](../../yql/reference/syntax/batch-update.md) и [BATCH DELETE FROM](../../yql/reference/syntax/batch-delete.md)) выполняются вне транзакции. Запрос может состоять только из одной инструкции пакетного изменения. При возникновении ошибки изменения инструкции не откатываются. | ||
|
|
||
| #### Сводная таблица |
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.
И этот заголовок бы убрал.
|
|
||
| Если для запроса не задан [режим транзакции](../transactions.md#modes), {{ ydb-short-name }} автоматически управляет его поведением. Такой режим называется неявной транзакцией. | ||
|
|
||
| В этом режиме {{ ydb-short-name }} на основе запроса определяет, выполнить его вне транзакции или обернуть в транзакцию с режимом *Serializable*. Режим неявной транзакции является универсальным для выполнения запроса, так как поддерживает инструкции любого вида с определённым поведением, описанным ниже. |
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.
Я бы добавил объяснение. Грубо говоря, YDB не поддерживает транзакционный DDL, поэтому появляется описание работы неявных транзакций.
|
|
||
| | Тип инструкции | Обработка неявной транзакции | Поддержка нескольких инструкций | Откат при ошибке | | ||
| |----------------|---------------------------------------------------|---------------------------------|-----------------------| | ||
| | DDL | Вне транзакции | Да (только DDL) | Нет | |
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.
Мне не нравится фраза "Вне транзакции". Больше нравится "Каждый DDL statement в своей транзакции"
…tform#21324) Co-authored-by: Ivan Blinkov <ivan@blinkov.ru>
…tform#21324) Co-authored-by: Ivan Blinkov <ivan@blinkov.ru>
Changelog entry
...
Changelog category
Description for reviewers
...