Prepared transactionをプリペアドトランザクションに統一 - #3706
Open
noborus wants to merge 2 commits into
Open
Conversation
- #2490の対応です。 - prepared statementの方は別に出す予定です。
KenichiroTanaka
requested changes
Jul 26, 2026
KenichiroTanaka
left a comment
KenichiroTanaka
left a comment
Contributor
There was a problem hiding this comment.
対応ありがとうございました。
いくつか気づいたところがあったのでみていただいてもいいでしょうか。
| 明示的あるいは暗黙的にロールバックされたトランザクションは、決してデコードされません。 | ||
| 成功したセーブポイントは、実行された順番にセーブポイントが実行されたトランザクションの中に折り込まれます。 | ||
| <command>PREPARE TRANSACTION</command>を使用して2相コミット用に準備されたトランザクションも、デコードに必要な出力プラグインコールバックが提供されていればデコードされます。 | ||
| <command>PREPARE TRANSACTION</command>を使用して2相コミット用にプリペアドトランザクションも、デコードに必要な出力プラグインコールバックが提供されていればデコードされます。 |
Contributor
There was a problem hiding this comment.
準備された、であればつながっていた日本語がプリペアドにすると少し不自然に感じました。
「2相コミット用にプリペアドトランザクションも」
は
「2相コミット用のプリペアドトランザクションも」
としていただくのはどうでしょうか。
| 短命なプリペアドトランザクションは共有メモリとWALにのみ格納されます。 | ||
| チェックポイントをまたいだトランザクションは<filename>pg_twophase</filename>ディレクトリに記録されます。 | ||
| 現在準備済みのトランザクションは<link linkend="view-pg-prepared-xacts"><structname>pg_prepared_xacts</structname></link>を使用して検査できます。 | ||
| 現在プリペアドのトランザクションは<link linkend="view-pg-prepared-xacts"><structname>pg_prepared_xacts</structname></link>を使用して検査できます。 |
Contributor
There was a problem hiding this comment.
「現在プリペアドのトランザクションは」という表現が気になりました。
「現在のプリペアドトランザクションは」などとするのがいいでしょうか。
| <refpurpose>commit a transaction that was earlier prepared for two-phase commit</refpurpose> | ||
| --> | ||
| <refpurpose>二相コミット用に事前に準備されたトランザクションをコミットする</refpurpose> | ||
| <refpurpose>二相コミット用に事前にプリペアドトランザクションをコミットする</refpurpose> |
Contributor
There was a problem hiding this comment.
「論理レプリケーションワーカーによってプリペアドトランザクションは次のようなパターンを持ちます」
ですが、ちょっと文章が不自然になってしまいました。
「論理レプリケーションワーカーによるプリペアドトランザクションは」
とするのはどうでしょうか。
| <refpurpose>cancel a transaction that was earlier prepared for two-phase commit</refpurpose> | ||
| --> | ||
| <refpurpose>二相コミット用に事前に準備されたトランザクションを取り消す</refpurpose> | ||
| <refpurpose>二相コミット用に事前にプリペアドトランザクションを取り消す</refpurpose> |
Contributor
There was a problem hiding this comment.
こちらも「二相コミット用の事前にプリペアドトランザクションを取り消す」としていただくのはどうでしょうか。
Member
Author
There was a problem hiding this comment.
commit_prepared.sgmlと合わせて、prepared transactionではなかったので、元に戻して修正しました。
- 指摘されたプリペアドトランザクションにしたことで変になっていた箇所の修正 - 二相コミットを2相コミットに修正 - 「prepared transaction」でなかった箇所は「準備されたトランザクション」に戻す
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.