Re: tsvector work with citext

From: Teodor Sigaev <teodor(at)sigaev(dot)ru>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "David E(dot) Wheeler" <david(at)kineticode(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Me Chai Swee <mech(at)me(dot)com>
Subject: Re: tsvector work with citext
Date: 2015-09-17 13:17:58
Message-ID: 55FABD86.3010109@sigaev.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> Oversight, I suppose. is_text_type() was introduced by
>
> commit 635aaab278afc1af972a4b6a55ff632ab763505d
> Author: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> Date: Tue Apr 8 18:20:29 2008 +0000
>
> Fix tsvector_update_trigger() to be domain-friendly: it needs to allow all
> the columns it works with to be domains over the expected type, not just
> exactly the expected type. In passing, fix ts_stat() the same way.
> Per report from Markus Wollny.

I'm wrong, in this commit it was just renamed. It was originally coded by me.
But it's still oversight.

--
Teodor Sigaev E-mail: teodor(at)sigaev(dot)ru
WWW: http://www.sigaev.ru/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2015-09-17 13:33:54 Re: planstate_tree_walker
Previous Message Teodor Sigaev 2015-09-17 12:51:53 Re: tsvector work with citext