From: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | emre(at)hasegeli(dot)com |
Cc: | kgrittn(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org, tgl(at)sss(dot)pgh(dot)pa(dot)us, andreas(at)proxel(dot)se, teodor(at)sigaev(dot)ru, robertmhaas(at)gmail(dot)com, kgrittn(at)ymail(dot)com, Jim(dot)Nasby(at)bluetreble(dot)com, mail(at)joeconway(dot)com |
Subject: | Re: Floating point comparison inconsistencies of the geometric types |
Date: | 2016-11-11 02:04:21 |
Message-ID: | 20161111.110421.122299480.horiguchi.kyotaro@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello,
> > Returning to the issue, the following query should give you the
> > expected result.
> >
> > SELECT name, #thepath FROM iexit ORDER BY name COLLATE "C", 2;
>
> Yes, I have worked around it like this. What I couldn't understand is
> how my patch can cause this regression. How is it passes on master
> without COLLATE?
Ah, sorry, I understand that *you* added the COLLATE. Revering
select_views.sql/out to master actually causes a regression error.
The reason for that is that you forgot to edit an alternative
exptect file, which matches for en_US locale.
But the test doesn't for collation and the only difference
between the alternative expect files comes from the difference of
collation for the query. "the workaround" seems to be the right
way to do it. I recommend rather to leave the workaroud as it is
and remove select_views_1.out from the "expected" directory.
Aside from that, I'd like to comment this patch on other points
later.
regards,
--
Kyotaro Horiguchi
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2016-11-11 02:25:47 | Re: Unlogged tables cleanup |
Previous Message | Michael Paquier | 2016-11-11 00:56:21 | Re: Shared memory estimation for postgres |