Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ddl.sgmlのPostgreSQL 15.0対応です。 #2573

Merged
merged 3 commits into from
Mar 27, 2023

Conversation

tatsuo-ishii
Copy link
Contributor

No description provided.

@KenichiroTanaka
Copy link
Contributor

ひととおり確認しましたが気づくところがありませんでした。
少し量も多いので見落としがあるかもしれません。どなたか見ていただけるとありがたいです。

デフォルトでは、2つのNULL値はこの比較では等しくないとみなされます。
つまり、一意性制約が存在する場合でも、制約された列の少なくとも1つにNULL値を含む重複行を格納できます。
この動作は、次のように句<literal>NULLS NOT DISTINCT</literal>を追加することで変更できます。
一般的には、制約に含まれるすべての列の値が等しい複数の行が表の中にある場合に、一意性制約に違反します。
Copy link
Contributor

Choose a reason for hiding this comment

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

この前の1113行の「表制約」とここだけ「表」になっていますが、他は「テーブル」となっているので「テーブル」で良いと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

おお、そうですね。

@@ -1707,7 +1707,7 @@ CREATE TABLE posts (
the column <literal>tenant_id</literal> to null, but that column is still
required as part of the primary key.
-->
《機械翻訳》カラムが指定されていない場合、外部キーもカラム<literal>tenant_id</literal>をnullに設定しますが、このカラムはプライマリ・キーの一部として必要です
カラムが指定されていない場合、外部キーも<literal>tenant_id</literal>列をnullに設定しますが、このカラムは主キーの一部として必要です
Copy link
Contributor

Choose a reason for hiding this comment

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

逆にこちらは「列が指定されていない場合」で良いかと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

はい、修正します。

Copy link
Contributor

Choose a reason for hiding this comment

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

5268行で このような継承関係の動的追加、動的削除は、継承関係をテーブル分割(<xref linkend="ddl-partitioning"/>を参照)に使用している場合に有用です。となっていますが、
ddl-partitioning の方のタイトルが、テーブルのパーティショニングになっているので、テーブルパーティショニング で良いと思います。
関連して、 #2227 の問題が残っていますが、ddl.sgmlは一括で変更して合わせればよさそうです(これは後でも大丈夫です)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

5268行で このような継承関係の動的追加、動的削除は、継承関係をテーブル分割(を参照)に使用している場合に有用です。となっていますが、
ddl-partitioning の方のタイトルが、テーブルのパーティショニングになっているので、テーブルパーティショニング で良いと思います。

そうします。

関連して、 #2227 の問題が残っていますが、ddl.sgmlは一括で変更して合わせればよさそうです(これは後でも大丈夫です)

では後でということで。

Copy link
Contributor Author

@tatsuo-ishii tatsuo-ishii left a comment

Choose a reason for hiding this comment

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

レビューありがとうございます。

デフォルトでは、2つのNULL値はこの比較では等しくないとみなされます。
つまり、一意性制約が存在する場合でも、制約された列の少なくとも1つにNULL値を含む重複行を格納できます。
この動作は、次のように句<literal>NULLS NOT DISTINCT</literal>を追加することで変更できます。
一般的には、制約に含まれるすべての列の値が等しい複数の行が表の中にある場合に、一意性制約に違反します。
Copy link
Contributor Author

Choose a reason for hiding this comment

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

おお、そうですね。

@@ -1707,7 +1707,7 @@ CREATE TABLE posts (
the column <literal>tenant_id</literal> to null, but that column is still
required as part of the primary key.
-->
《機械翻訳》カラムが指定されていない場合、外部キーもカラム<literal>tenant_id</literal>をnullに設定しますが、このカラムはプライマリ・キーの一部として必要です
カラムが指定されていない場合、外部キーも<literal>tenant_id</literal>列をnullに設定しますが、このカラムは主キーの一部として必要です
Copy link
Contributor Author

Choose a reason for hiding this comment

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

はい、修正します。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

5268行で このような継承関係の動的追加、動的削除は、継承関係をテーブル分割(を参照)に使用している場合に有用です。となっていますが、
ddl-partitioning の方のタイトルが、テーブルのパーティショニングになっているので、テーブルパーティショニング で良いと思います。

そうします。

関連して、 #2227 の問題が残っていますが、ddl.sgmlは一括で変更して合わせればよさそうです(これは後でも大丈夫です)

では後でということで。

デフォルトでは、2つのNULL値はこの比較では等しくないとみなされます。
つまり、一意性制約が存在する場合でも、制約された列の少なくとも1つにNULL値を含む重複行を格納できます。
この動作は、次のように句<literal>NULLS NOT DISTINCT</literal>を追加することで変更できます。
一般的には、制約に含まれるすべての列の値が等しい複数の行が表の中にある場合に、一意性制約に違反します。
Copy link
Contributor

Choose a reason for hiding this comment

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

制約に含まれるすべての列の値が等しい複数の行が表の中にある場合

この表もテーブルで良いと書いたつもりがわかりづらい書き方になってました。すみません。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

失礼しました。修正しました。

@noborus
Copy link
Contributor

noborus commented Mar 27, 2023

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

@noborus noborus merged commit ff41820 into pgsql-jp:doc_ja_15 Mar 27, 2023
@tatsuo-ishii tatsuo-ishii deleted the ddl150 branch August 18, 2023 07:44
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.

3 participants