Re: proposal: variadic argument support for least, greatest function - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: proposal: variadic argument support for least, greatest function
Date
Msg-id 67d2ea79-a6e2-e784-810b-b801b3afff1a@2ndQuadrant.com
Whole thread Raw
In response to Re: Re: proposal: variadic argument support for least, greatestfunction  (David Steele <david@pgmasters.net>)
Responses Re: proposal: variadic argument support for least, greatest function
List pgsql-hackers
On 3/4/19 9:39 AM, David Steele wrote:
> On 3/1/19 3:59 AM, Chapman Flack wrote:
>> The following review has been posted through the commitfest application:
>> make installcheck-world:  tested, passed
>> Implements feature:       tested, passed
>> Spec compliant:           not tested
>> Documentation:            tested, passed
>>
>> For completeness, I'll mark this reviewed again. It passes
>> installcheck-world, the latest patch addresses the suggestions from
>> me, and is improved on the code-structure matters that Tom pointed
>> out. I don't know if it will meet Tom's threshold for desirability
>> overall, but that sounds like a committer call at this point, so I'll
>> change it to RfC.
>
> Both committers who have looked at this patch (Tom, and Andres in his
> patch roundup [1]) recommend that it be rejected.
>
> If no committer steps up in the next week I think we should mark it as
> rejected.
>
>

Having reviewed the thread, I'm with Andres and Tom. Maybe though we
should have a note somewhere to the effect that you can't use VARIADIC
with these.


cheers


andrew

-- 
Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: "Karl O. Pinc"
Date:
Subject: Re: Patch to document base64 encoding
Next
From: Alvaro Herrera
Date:
Subject: Re: [HACKERS] CLUSTER command progress monitor