On Wed, Dec 7, 2011 at 10:09 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> There's some stuff that's debatable according to this criterion --- in
> particular, I wondered whether it'd be worth having a fast path for
> bttextcmp, especially if we pre-tested the collate_is_c condition and
> had a separate version that just hardwired the memcmp code path. (The
> idea of doing that was one reason I insisted on collation being known at
> the setup step.) But it would still have to be prepared for detoasting,
> so in the end I was unenthused. Anyone who feels like testing could try
> to prove me wrong about it though.
I think that'd definitely be worth investigating (although I'm not
sure I have the time to do it myself any time real soon).
>> Are you planning to do anything about #2 or #3?
>
> I am willing to do #2, but not right now; I feel what I need to do next
> is go review SPGist.
Yeah, makes sense. That one seems likely to be a challenge to absorb.
> I don't believe that #2 blocks progress on #3
> anyway. I think #3 is in Peter's court, or yours if you want to do it.
>
> (BTW, I agree with your comments yesterday about trying to break down
> the different aspects of what Peter did, and put as many of them as we
> can into the non-inlined code paths.)
Cool. Peter, can you rebase your patch and integrate it into the
sortsupport framework that's now committed?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
