From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael.paquier@gmail.com>
Cc: PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Subject: Re: ALTER COLUMN TYPE vs. domain constraints
Date: Wed, 01 Nov 2017 13:36:46 -0400
Message-ID: <9888.1509557806@sss.pgh.pa.us> (raw)
In-Reply-To: <CAB7nPqRhe+t=e-JJLcsmbQaZUbrVJhux_8OtdoFevdyCREYPUg@mail.gmail.com>
References: <30656.1509128130@sss.pgh.pa.us>
<CAB7nPqRhe+t=e-JJLcsmbQaZUbrVJhux_8OtdoFevdyCREYPUg@mail.gmail.com>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
Michael Paquier <michael.paquier@gmail.com> writes:
> On Fri, Oct 27, 2017 at 11:15 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> We could consider back-patching the attached to cover this, but
>> I'm not entirely sure it's worth the trouble, because I haven't
>> thought of any non-silly use-cases in the absence of domains
>> over composite. Comments?
> There are no real complaints about the current behavior, aren't there?
> So only patching HEAD seems enough to me.
Yeah, we can leave it till someone does complain.
> You have added a comment on the constraint to make sure that it
> remains up on basically this ALTER TYPE. Querying pg_obj_description
> would make sure that the comment on the constraint is still here.
Done.
> +RebuildDomainConstraintComment(AlteredTableInfo *tab, int pass, Oid objid,
> + List *domname, char *conname)
> There is much duplication with RebuildConstraintComment. Why not
> grouping both under say RebuildObjectComment()?
True. I'd originally expected the code to differ more, but we can
merge these easily enough.
> I would think about
> having cmd->objtype and cmd->object passed as arguments, and then
> remove rel and domname from the existing arguments.
Doing it like that requires the callers to do work (to prepare the object
ID data structure) that's wasted if there's no comment, which most often
there wouldn't be, I suspect. Seems better to just pass the info the
caller does have and let this function do the rest.
Pushed, thanks for reviewing!
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, michael.paquier@gmail.com
Subject: Re: ALTER COLUMN TYPE vs. domain constraints
In-Reply-To: <9888.1509557806@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox