Re: BUG #16333: position() function not equivalent to strpos()function when comparing citext - Mailing list pgsql-bugs

From Peter Eisentraut
Subject Re: BUG #16333: position() function not equivalent to strpos()function when comparing citext
Date
Msg-id 9817af55-fbe4-b342-b641-637f1308ef96@2ndquadrant.com
Whole thread Raw
In response to Re: BUG #16333: position() function not equivalent to strpos() function when comparing citext  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: BUG #16333: position() function not equivalent to strpos() function when comparing citext
List pgsql-bugs
On 2020-04-02 03:04, Tom Lane wrote:
> PG Bug reporting form <noreply@postgresql.org> writes:
>> SELECT position('foo'::citext IN 'Foobar'::citext) =
>> strpos('Foobar'::citext, 'foo'::citext) as "positionEqualsStrpos";
> 
>> Citext does not create an overload for position() supporting citext
>> parameters, therefore position(a in b) always runs case-sensitive.
> 
> Well, the citext documentation specifies which functions have
> case-insensitive mappings.  strpos() is listed, position() is not,
> so I'd say it's acting precisely as documented.

Arguably, there is a misdesign here, however.  Any function that does 
some kind of text-in-text search where citext could plausbily offer 
case-insensitive behavior will automatically fall back to the 
case-sensitive version if citext doesn't offer its own variant.  The fix 
would technically need to be that citext offers its own variant of every 
potential such function, which is clearly not possible, or that casts 
between text and citext are more restricted, which would make citext 
nearly unusable.

Doesn't seem fixable.  Collations are probably a better way of dealing 
with this.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-bugs by date:

Previous
From: Tom Lane
Date:
Subject: Re: BUG #16338: casting interval '24h' to time returns '00:00:00' instead of '24:00:00'
Next
From: Tom Lane
Date:
Subject: Re: BUG #16333: position() function not equivalent to strpos() function when comparing citext