From: | Andreas Karlsson <andreas(at)proxel(dot)se> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Parallel safety tagging of extension functions |
Date: | 2016-06-01 22:36:33 |
Message-ID: | 574F6371.3040802@proxel.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 05/25/2016 03:32 AM, Tom Lane wrote:
> Andreas Karlsson <andreas(at)proxel(dot)se> writes:
>> On 05/25/2016 02:41 AM, Robert Haas wrote:
>>> I'd rather extend see us ALTER AGGREGATE to do this.
>
>> Wouldn't that prevent this from going into 9.6? I do not think changing
>> ALTER AGGREGATE is 9.6 materials.
>
> Well, it's debatable --- but if the patch to do it is small and the
> alternatives are really ugly, that would be an acceptable choice IMO.
> Certainly we'd want to add that capability eventually anyway.
Looked at this quickly and I do not think adding it would be what I
consider a small patch since we would essentially need to copy the
validation logic from DefineAggregate and AggregateCreate and modify it
to fit the alter case. I am leaning towards either either leaving the
aggregate functions alone or updating the catalog manually.
Andreas
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2016-06-01 23:39:01 | Re: PostmasterPid not marked with PGDLLIMPORT |
Previous Message | Andres Freund | 2016-06-01 22:33:18 | Re: Perf Benchmarking and regression. |