Skip to content

Conversation

@higuchi-daisuke
Copy link
Contributor

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Japanese translation of the PostgreSQL pg_dumpall command documentation for version 18.0. The changes improve translation quality by removing machine-translated text markers and updating terminology to be more natural and consistent.

Key changes include:

  • Standardizing terminology: "統計処理" → "統計情報" (statistics), "データの定義" → "データ定義" (data definitions)
  • Removing machine translation markers (《機械翻訳》, 《マッチ度》) and replacing with polished Japanese text
  • Improving clarity and naturalness of Japanese documentation throughout

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

スキーマ(データの定義)やデータは含まれません。
テーブル、実体化ビュー(Materialized View)、外部テーブル、インデックスの統計処理がダンプされます。
統計情報のみをダンプし、スキーマ(データ定義)とデータはダンプしません。
テーブル、マテリアライズドビュー、外部テーブル、そしてインデックスの統計情報がダンプされます。
Copy link

Copilot AI Nov 15, 2025

Choose a reason for hiding this comment

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

[nitpick] 連続する項目の接続詞が不統一です。「外部テーブル、そしてインデックス」となっていますが、日本語の技術文書では通常「そして」は使わず、「外部テーブル、およびインデックス」または「外部テーブル、インデックス」とするのが一般的です。他の箇所(139行目など)では「と」が使われており、一貫性のため「外部テーブル、およびインデックス」または「外部テーブルとインデックス」とすることを推奨します。

Suggested change
テーブル、マテリアライズドビュー、外部テーブル、そしてインデックスの統計情報がダンプされます
テーブル、マテリアライズドビュー、外部テーブル、およびインデックスの統計情報がダンプされます

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちらは修正した方が良いと考え、修正しました。

<option>--statistics</option>が指定されている場合、<command>pg_dumpall</command>は出力されるダンプファイルにほとんどのオプティマイザの統計情報を含めます
しかし、<xref linkend="sql-createstatistics"/>で明示的に作成された統計情報や、拡張により追加された独自統計情報など、一部の統計情報は含まれない場合があります
そのため、最適な性能を発揮するためには、ダンプファイルからリストアした後に各データベースで<command>ANALYZE</command>を実行すると効果的な場合があります
また、<command>vacuumdb -a -z</command>を実行して、全てのデータベースを解析することができます
Copy link

Copilot AI Nov 15, 2025

Choose a reason for hiding this comment

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

[nitpick] 「全てのデータベースを解析する」という表現は、文脈的に「ANALYZE」コマンドを実行することを指しているため、「解析する」よりも「ANALYZEする」または「統計情報を収集する」の方が技術的に正確です。他の箇所では「ANALYZE」とカタカナ表記されているため、一貫性のため「全てのデータベースをANALYZEする」とすることを推奨します。

Suggested change
また、<command>vacuumdb -a -z</command>を実行して、全てのデータベースを解析することができます
また、<command>vacuumdb -a -z</command>を実行して、全てのデータベースをANALYZEすることができます

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちらは原文が「You can also run vacuumdb -a -z to analyze all databases.」となっております。
コマンドの"ANALYZE"を想定したものでもなさそうなので、通常通り訳した方が良いと考え、修正せずそのままにします。

@higuchi-daisuke
Copy link
Contributor Author

copilotの指摘事項を確認して対応しました。

Copy link
Member

@noborus noborus left a comment

Choose a reason for hiding this comment

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

確認しました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants