pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Jehan-Guillaume de Rorthais <jgdr@dalibo.com>
To: Peter Geoghegan <pg@bowt.ie>
Cc: Thomas Munro <thomas.munro@gmail.com>
Cc: Роман Литовченко <roman.lytovchenko@gmail.com>
Cc: PostgreSQL mailing lists <pgsql-bugs@lists.postgresql.org>
Subject: Re: BUG #15285: Query used index over field with ICU collation in some cases wrongly return 0 rows
Date: Tue, 18 Aug 2020 18:02:46 +0200
Message-ID: <20200818180246.04e47f8e@firost> (raw)
In-Reply-To: <CAH2-Wz=dN99ptML=DYnNt+byRBA8cnbHiHh=Ei7a0NPRQ89CYw@mail.gmail.com>
References: <153201618542.1404.3611626898935613264@wrigleys.postgresql.org>
	<CAH2-Wz=TGN_03qBmqgFXVhw6Kbfys-EBz4MJunzRq+RB5gL7WA@mail.gmail.com>
	<CAH2-Wzkiue6PQzneBJsuDnYdUVMa5jnevVJXXkV3hR0oboOSbQ@mail.gmail.com>
	<CAEepm=1-f3OuO_zA83C0x8dLFqMA+Q=RGUkyMRp4WXxjSrvbww@mail.gmail.com>
	<20200610002933.6a6d482b@firost>
	<20200612184055.205f0159@firost>
	<20200613004322.3bcc37f6@firost>
	<CAH2-Wz=dN99ptML=DYnNt+byRBA8cnbHiHh=Ei7a0NPRQ89CYw@mail.gmail.com>

Hi Peter,

On Tue, 11 Aug 2020 21:14:03 -0700
Peter Geoghegan <pg@bowt.ie> wrote:

> On Fri, Jun 12, 2020 at 3:43 PM Jehan-Guillaume de Rorthais
> <jgdr@dalibo.com> wrote:
> > Please, find in attachment the second version of the patch, with a
> > regression test.
> 
> Is it possible to fix this by making the existing
> HAVE_UCOL_STRCOLLUTF8 test more conservative about ICU version? IOW,
> by making it only use ucol_strcollUTF8() on versions that are known to
> not be affected by this bug?

I might be missing something, but according to my tests back in June, the bug
exists in both ucol_strcollUTF8()/ucol_strcoll() and is still affecting the very
last version of ICU (67.1).

That's why my patch replaces both functions altogether using ucol_strcollIter
as replacement.

> This related ICU bug describes an issue affecting only versions 53/54:
> 
> https://unicode-org.atlassian.net/browse/ICU-11388

This bug is related to ICU4J, not ICU4C. AS far as I understand, this was
related to a bad variable type when porting the code to java. Do I miss
something?

Regards,





view thread (21+ messages)  latest in thread

Message-ID: <20200818180246.04e47f8e@firost>
Permalink:  ../20200818180246.04e47f8e@firost/
Also on:    postgresql.org/message-id/20200818180246.04e47f8e@firost

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: jgdr@dalibo.com, pg@bowt.ie, thomas.munro@gmail.com, roman.lytovchenko@gmail.com, pgsql-bugs@lists.postgresql.org
  Subject: Re: BUG #15285: Query used index over field with ICU collation in some cases wrongly return 0 rows
  In-Reply-To: <20200818180246.04e47f8e@firost>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox