> 30 авг. 2019 г., в 16:44, Andrey Borodin <x4...@yandex-team.ru> написал(а):
> 
> 
> How do you think, should I supply GiST-build patch with docs and tests and 
> add it to CF? Or do we need more design discussion before?

PFA v2: now sort support is part of opclass.
There's a problem with Z-ordering NaN which causes regression tests to fail. So 
I decided not to add patch to CF (despite having few minutes to do so).
How to correctly Z-order NaN? So that it would be consistent with semantics of 
union() and consistent() functions. If one of values is NaN, then we consider 
all it's bits to be 1?

BTW patch uses 
union {
        float f;
        uint32 i;
    }
I hope it's OK, because AFAIK we do not have non-IEEE-754 platforms now.

Thanks!

Best regards, Andrey Borodin.

Attachment: v2-0001-Add-sort-support-for-point-gist_point_sortsupport.patch
Description: Binary data

Attachment: v2-0002-Implement-GiST-build-using-sort-support.patch
Description: Binary data

Reply via email to