Re: tablecmds.c/MergeAttributes() cleanup

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: tablecmds.c/MergeAttributes() cleanup
Date: 2023-10-05 15:49:24
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On 19.09.23 15:11, Peter Eisentraut wrote:
> Here is an updated version of this patch set.  I resolved some conflicts
> and addressed this comment of yours.  I also dropped the one patch with
> some catalog header edits that people didn't seem to particularly like.
> The patches that are now 0001 through 0004 were previously reviewed and
> just held for the not-null constraint patches, I think, so I'll commit
> them in a few days if there are no objections.
> Patches 0005 through 0007 are also ready in my opinion, but they haven't
> really been reviewed, so this would be something for reviewers to focus
> on.  (0005 and 0007 are trivial, but they go to together with 0006.)
> The remaining 0008 and 0009 were still under discussion and contemplation.

I have committed through 0007, and I'll now close this patch set as
"Committed", and I will (probably) bring back the rest (especially 0008)
as part of a different patch set soon.

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Julien Rouhaud 2023-10-05 16:02:15 Re: Good News Everyone! + feature proposal
Previous Message Peter Eisentraut 2023-10-05 15:46:46 Re: Remove distprep