From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | "Kevin Grittner" <Kevin(dot)Grittner(at)wicourts(dot)gov> |
Cc: | greg(at)2ndquadrant(dot)com, pgsql-hackers(at)postgresql(dot)org, oleg(at)sai(dot)msu(dot)su, teodor(at)sigaev(dot)ru |
Subject: | Re: tsearch parser inefficiency if text includes urls or emails - new version |
Date: | 2009-12-08 16:17:19 |
Message-ID: | 200912081717.19531.andres@anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tuesday 08 December 2009 17:15:36 Kevin Grittner wrote:
> Andres Freund <andres(at)anarazel(dot)de> wrote:
> > Could you show your testcase?
Will hopefully look into this later.
> > I dont see why it could get slower?
> I don't either. The best I can tell, following the pointer from
> orig to any of its elements seems to be way more expensive than I
> would ever have guessed. The only thing that seemed to improve the
> speed was minimizing that by using a local variable to capture any
> element referenced more than once. (Although, there is overlap
> between the timings for the original patch and the one which seemed
> a slight improvement; I would need to do more testing to really rule
> out noise and have complete confidence that my changes actually are
> an improvement on the original patch.)
>
> Perhaps it is some quirk of using 32 bit pointers on the 64 bit AMD
> CPU? (I'm looking forward to testing this today on a 64 bit build
> on an Intel CPU.)
Did you test that with a optimized build?
Andres
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2009-12-08 16:40:31 | Re: tsearch parser inefficiency if text includes urls or emails - new version |
Previous Message | Kevin Grittner | 2009-12-08 16:15:36 | Re: tsearch parser inefficiency if text includes urls or emails - new version |