Re: ICU locales and text/char(n) SortSupport on Windows

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: Noah Misch <noah(at)leadboat(dot)com>
Cc: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: ICU locales and text/char(n) SortSupport on Windows
Date: 2017-09-21 17:03:04
Message-ID: CAH2-WzkLJuV5Gocq_CGJDh8jS7nx=-oyRcNp6b16aeiQ8bxY9A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Sep 20, 2017 at 11:05 PM, Noah Misch <noah(at)leadboat(dot)com> wrote:
> This is currently a v10 open item, but I think it doesn't qualify for that
> treatment. It's merely an opportunity for optimization, albeit an
> attractively-simple one.

Fair enough.

This is clearly an omission that was made in 41839b7ab, the commit
that added/fixed Windows support for ICU. Hopefully the omission can
be corrected for v10. I see this as a surprising behavior for users,
since ICU locales on all other platforms *will* have much faster
sorting than libc locales (often more than 3x faster). This is mostly
because ICU brings back abbreviated keys. 3x - 5x faster is more of a
qualitative difference than a quantitative one, IMHO.

That having been said, I'm certainly not going to insist on a v10 fix
for this issue.

--
Peter Geoghegan

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Dilip Kumar 2017-09-21 17:04:48 Re: Effect of changing the value for PARALLEL_TUPLE_QUEUE_SIZE
Previous Message Dave Cramer 2017-09-21 17:01:10 Re: PSA: don't be in a hurry to update to XCode 9.0