On Jul 29, 2008, at 15:35, Tom Lane wrote:
> BTW, I did confirm (by temporarily hacking up TypeCategory()) that
> causing citext to appear to be of STRING category eliminates the need
> for the extra || operator and quote_literal() function that are in
> the current citext code. So the proposed solution really will work.
Nice! Does it also allow all of the casts to and from text to
implicitly work, e.g,
SELECT 4::int8::citext = '4' AS t;
SELECT '4'::citext::int8 = 4 AS t;
SELECT 4::bigint::citext = '4' AS t;
SELECT '4'::citext::bigint = 4 AS t;
> You might still want to keep the aliases in cases where the point is
> to
> have the function or operator output resolve as citext rather than
> text.
> I'm not sure how many of these cases that's really important for,
> though.
Not many, I should think.
Thanks,
David