On 05.03.25 13:56, Alvaro Herrera wrote:
On 2025-Mar-03, Suraj Kharage wrote:
Thanks Alvaro for the review and fixup patch.
I agree with your changes and merged that into the main patch along with a
couple of other changes.
Please find attached v6 for further review.
Thanks, I have pushed this. I made some changes to the tests, first by
renaming the tables to avoid too generic names, and second to try and
exercise everything about once.
A patch in the NOT ENFORCED constraints patch series proposes to
refactor some of the code added by this patch series ([0] patch
v18-0001). I noticed that the code paths from this patch series do not
call InvokeObjectPostAlterHook() or CacheInvalidateRelcache() when a
constraint is altered. Was this intentional? If not, I can fix it as
part of that other patch, just wanted to check here.
[0]:
https://www.postgresql.org/message-id/caaj_b97ahsjgwhaurqi1jdwsjzd_ygwejqqvq_ddo8dycnn...@mail.gmail.com
