From: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [COMMITTERS] pgsql: Add citext_pattern_ops for citext contrib module |
Date: | 2017-09-19 14:28:07 |
Message-ID: | CANP8+j+Eua-BooiavjJ=-srC46m088y3yRnOmFDyd-iXi6f3pw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 19 September 2017 at 15:22, Andrew Dunstan
<andrew(dot)dunstan(at)2ndquadrant(dot)com> wrote:
>
>
> On 09/19/2017 08:35 AM, Andrew Dunstan wrote:
>> Add citext_pattern_ops for citext contrib module
>>
>> This is similar to text_pattern_ops.
>>
>
> This seems to have upset a number or animals in the buildfarm.
>
> I could create a third output file, but I am seriously questioning the
> point of all this, if we are prepared to accept any result from these
> functions and operators, depending on locale. Maybe it would be better
> simply to test that the result is not null and have done with it. Thoughts?
It makes sense to have a fully detailed output in at least one
parameter setting.
How about use the new psql feature for \if to skip tests if the local
is different to the one for which we have detailed test results?
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-09-19 15:02:00 | Re: Re: [COMMITTERS] pgsql: Add citext_pattern_ops for citext contrib module |
Previous Message | Andrew Dunstan | 2017-09-19 14:22:52 | Re: [COMMITTERS] pgsql: Add citext_pattern_ops for citext contrib module |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2017-09-19 14:30:42 | Re: src/test/subscription/t/002_types.pl hanging on particular environment |
Previous Message | Andrew Dunstan | 2017-09-19 14:22:52 | Re: [COMMITTERS] pgsql: Add citext_pattern_ops for citext contrib module |