Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit72eab84

Browse files
committed
Correctly mark pg_subscription.subslotname as nullable.
Due to the layout of this catalog, subslotname has to be explicitlymarked BKI_FORCE_NULL, else initdb will default to the assumptionthat it's non-nullable. Since, in fact, CREATE/ALTER SUBSCRIPTIONwill store null values there, the existing marking is just wrong,and has been since this catalog was invented.We haven't noticed because not much in the system actually dependson attnotnull being truthful. However, JIT'ed tuple deconstructiondoes depend on that in some cases, allowing crashes or wrong answersin queries that inspect pg_subscription. Commit9de77b5 quiteaccidentally exposed this on the buildfarm members that force JITactivation.Back-patch to v13. The problem goes further back, but we cannotforce initdb in released branches, so some klugier solution willbe needed there. Before working on that, push this simple fixto try to get the buildfarm back to green.Discussion:https://postgr.es/m/4118109.1595096139@sss.pgh.pa.us
1 parent4d3db13 commit72eab84

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

‎doc/src/sgml/catalogs.sgml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7529,7 +7529,8 @@ SCRAM-SHA-256$<replaceable>&lt;iteration count&gt;</replaceable>:<replaceable>&l
75297529
</para>
75307530
<para>
75317531
Name of the replication slot in the upstream database (also used
7532-
for the local replication origin name)
7532+
for the local replication origin name);
7533+
null represents <literal>NONE</literal>
75337534
</para></entry>
75347535
</row>
75357536

‎src/include/catalog/catversion.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,6 @@
5353
*/
5454

5555
/*yyyymmddN */
56-
#defineCATALOG_VERSION_NO202007181
56+
#defineCATALOG_VERSION_NO202007192
5757

5858
#endif

‎src/include/catalog/pg_subscription.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ CATALOG(pg_subscription,6100,SubscriptionRelationId) BKI_SHARED_RELATION BKI_ROW
5656
textsubconninfoBKI_FORCE_NOT_NULL;
5757

5858
/* Slot name on publisher */
59-
NameDatasubslotname;
59+
NameDatasubslotnameBKI_FORCE_NULL;
6060

6161
/* Synchronous commit setting for worker */
6262
textsubsynccommitBKI_FORCE_NOT_NULL;

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp