Re: Re: [COMMITTERS] pgsql: Add citext_pattern_ops for citext contrib module

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

In response to

Browse pgsql-committers by date

  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

Browse pgsql-hackers by date

  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