| From: | Jeff Davis <pgsql(at)j-davis(dot)com> |
|---|---|
| To: | Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Noah Misch <noah(at)leadboat(dot)com>, akapila(at)postgresql(dot)org, nisha(dot)moond412(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: REASSIGN OWNED vs. relisshared dep on !relisshared |
| Date: | 2026-07-14 20:50:00 |
| Message-ID: | 48531fdd144b17fa10b02adae33705de3ada8ea3.camel@j-davis.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, 2026-07-13 at 17:04 +0530, Dilip Kumar wrote:
> PFA a patch for skipping subscriptions belonging to different
> databases.
I suggest we add a few checks (or asserts) that subdbid==MyDatabaseId
when reading pg_subscription from anywhere other than the launcher.
Also, the REASSIGN OWNED path goes through
AlterSubscriptionOwner_oid(), whereas both the REASSIGN OWNED and ALTER
SUBSCRIPTION ... OWNER TO paths go through
AlterSubscriptionOwner_internal(). It would be more clear to add the
skipping logic in the former and an assert in the latter.
Other than that, LGTM.
We can also reconsider how we handle dependencies to address Noah's
concern, but this patch is good regardless.
Regards,
Jeff Davis
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tristan Partin | 2026-07-14 20:50:17 | Re: Grab bag of smaller OpenSSL fixups |
| Previous Message | Tomas Vondra | 2026-07-14 20:46:41 | Re: hashjoins vs. Bloom filters (yet again) |