Skip to content

tableに対してテーブルが入っていないところの見直し#3193

Merged
KenichiroTanaka merged 3 commits intopgsql-jp:doc_ja_16from
noborus:miss-table
Jan 5, 2025
Merged

tableに対してテーブルが入っていないところの見直し#3193
KenichiroTanaka merged 3 commits intopgsql-jp:doc_ja_16from
noborus:miss-table

Conversation

@noborus
Copy link
Member

@noborus noborus commented Dec 27, 2024

テーブルが抜けている箇所の修正。
information_schema.sgmlは記述を統一するように見直しました。

テーブルが抜けている箇所の修正。
information_schema.sgmlは記述を統一するように見直しました。
@noborus noborus added the version 16 version 16を対象 label Dec 27, 2024
Copy link
Contributor

@KenichiroTanaka KenichiroTanaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正いただいたところ以外の箇所なのですが1つだけ気づいてしまったので
見ていただけないでしょうか。

@@ -2546,7 +2546,7 @@ SELECT * FROM x, y, a, b, c WHERE something AND somethingelse;
an <function>fsync</function> at the end than by writing WAL.)
-->
こうすると、WALデータを処理する保管処理またはWAL送信処理にかかる時間がかからないことの他に、実際のところ、特定のコマンドをより高速にします。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

今回のレビュー対象外なのですが、
「WALデータを処理する保管処理またはWAL送信処理にかかる時間」
の箇所ですがarchiverは「保管処理」よりも「アーカイブ処理」の方が分かりやすかなと思います。
(直前の2534行目もそのようになっています。)
あと処理(process)も多く訳出されているため、
「WALデータをアーカイブまたはWAL送信処理するためにかかる時間」
くらいの訳とするのはどうでしょうか。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ちょっと見直して、
「こうすると、アーカイブやWAL送信にWALデータを処理する時間を避けることの他に、実際のところ、特定のコマンドをより高速にします。」としてみました。

ほんとは、複数ファイルにまたがっての修正の場合は、関係ない行は他のPRにした方がよいですね。

@KenichiroTanaka
Copy link
Contributor

対応ありがとうございます。
そうですね。
複数ファイル修正の時は別prにするようにします。

@KenichiroTanaka KenichiroTanaka merged commit d99b1c1 into pgsql-jp:doc_ja_16 Jan 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants