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 tableam.sgml documentation file for PostgreSQL version 18.0, replacing machine-translated text (marked with 《機械翻訳》) with properly translated Japanese content.

Key changes:

  • Improved translations of technical descriptions for table access method handlers
  • Enhanced clarity in explaining TableAmRoutine structure and callbacks
  • Refined explanations of handler function declarations and usage

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

戻り値のニーズはサーバの寿命になります。
これは通常、グローバルスコープで<literal>static const</literal>変数として定義することで実現されます。
関数の結果は<structname>TableAmRoutine</structname>型の構造体へのポインタでなければならず、そこにはテーブルアクセスメソッドを使用するためにコアコードが知る必要のあるすべてのことが含まれます。
戻り値はサーバが生存中必要で、通常これはグローバルスコープで<literal>static const</literal>変数として定義することで達成されます。
Copy link
Member

Choose a reason for hiding this comment

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

archive-modules.sgmlにも同じ文があったのですが、そちらはちょっと問題があるようなので、こちらで訳が決まったら修正しようと思います。

「lifetime」をどうするかですが、「存続期間」とすると他と統一されるかなと思います。
翻訳の候補を出します。

戻り値は、サーバの存続期間中は有効である必要があります。これは通常、グローバルスコープで<literal>static const</literal>変数として定義することによって達成されます。

@higuchi-daisuke
Copy link
Contributor Author

レビューありがとうございます。修正いたしました。

「lifetime」は確かに「存続期間」と訳している文が多いと思いましたのでそのように修正します。
また、いただいた候補についても読みやすく理解しやすいと感じ、他に案も思いつかなかったため、そのまま使用させていただきました。

@noborus
Copy link
Member

noborus commented Nov 17, 2025

レビューありがとうございます。修正いたしました。

「lifetime」は確かに「存続期間」と訳している文が多いと思いましたのでそのように修正します。 また、いただいた候補についても読みやすく理解しやすいと感じ、他に案も思いつかなかったため、そのまま使用させていただきました。

ありがとうございます。確認しました。

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