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 documentation for pg_dump.sgml to version 18.0, improving translation quality by removing machine translation artifacts and refining the Japanese text throughout.

  • Removes translation quality markers (《マッチ度[...]》 and 《機械翻訳》) from the documentation
  • Updates terminology for consistency (e.g., "統計処理" → "統計情報", contextual use of "バックアップ" vs "エクスポート")
  • Refines Japanese translations for better clarity and natural language flow

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

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.

確認しました。

sequence dataなのかsequence valueなのか、原文で使い分けているようには見えないのですが、誰も気にしてないのかな...

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

Choose a reason for hiding this comment

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

pg_dumpall.sgmlを変更したなら、こちらもです。

@higuchi-daisuke
Copy link
Contributor Author

レビューありがとうございます。
pg_dumpallにてcopilotからの指摘を対応した後、こちらを確認できていませんでした。
ご指摘いただいた点について修正しました。

Access privileges for the database itself are also dumped,
unless <option>&#45;-no-acl</option> is specified.
-->
<option>--create</option>では出力に、もしあるならデータベースのコメントも含まれます。また、あらゆる設定変数の設定、すなわち、このデーベースを対象としている<command>ALTER DATABASE ... SET ...</command>と<command>ALTER ROLE ... IN DATABASE ... SET ...</command>コマンドも含まれます。
Copy link
Member

Choose a reason for hiding this comment

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

今回とは関係ないところですみません。「デーベース」というtypoを見つけました。

Suggested change
<option>--create</option>では出力に、もしあるならデータベースのコメントも含まれます。また、あらゆる設定変数の設定、すなわち、このデータベースを対象としている<command>ALTER DATABASE ... SET ...</command>と<command>ALTER ROLE ... IN DATABASE ... SET ...</command>コマンドも含まれます。

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