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
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 |