Skip to content
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

fix for that i(ud) abort when it finds wp #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thawk105
Copy link
Contributor

shirakami で insert / update / delete_record が wp を見たとき、その操作の取り消し(warn_conflict_on_write_preserve) としていた。他操作との統一性、現仕様との整合性を考慮して、それらが wp を見たときにアボートするようにしました。

そのため、本修正がより適切であると考えています。
本テストがコケる事の再現性としては、
https://github.com/project-tsurugi/tsurugidb/actions/runs/6582842107
上記CIと、自身で動作させた野良ビルドでほぼ100%であることを確認しました。

当該修正に関して、shirakami では依然として dev ブランチにありまして、まだ master branch に反映していません。本修正が master に反映されると同時に、shirakami も master に反映しようと思います。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant