From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | obartunov(at)gmail(dot)com |
Cc: | pgsql-docs(at)postgresql(dot)org |
Subject: | Re: a little fix for text search |
Date: | 2016-11-12 21:37:05 |
Message-ID: | 6700.1478986625@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
Oleg Bartunov <obartunov(at)gmail(dot)com> writes:
> On Sat, Nov 12, 2016 at 11:49 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> I don't see why that stopped being appropriate? The point is that it
>> takes a raw text input which has to be re-parsed; that's still true
>> AFAICS.
> I mean that in the past we recommended to use subselect to avoid extra
> ts_headline() call, which now, at least at 9.6, it's obsoleted and two sql
> queries call ts_headline() exactly 5 times.
Oh, I see your point: commit 9118d03a8 fixed the planner so you don't get
extra evaluations of ts_headline() in this example. I think it's probably
still appropriate to warn that ts_headline() is expensive, but yes, the
specific example is obsolete.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2016-11-13 02:49:40 | Re: SET AUTOCOMMIT TO OFF is no longer supported |
Previous Message | Oleg Bartunov | 2016-11-12 21:01:46 | Re: a little fix for text search |