Re: [PATCH] Add sortsupport for range types and btree_gist - Mailing list pgsql-hackers

From Andrey M. Borodin
Subject Re: [PATCH] Add sortsupport for range types and btree_gist
Date
Msg-id F6394698-D6BD-469A-BEC4-2D23D348A324@yandex-team.ru
Whole thread Raw
In response to [PATCH] Add sortsupport for range types and btree_gist  (Christoph Heiss <christoph.heiss@cybertec.at>)
List pgsql-hackers

> On 25 Nov 2024, at 20:40, Bernd Helmle <mailings@oopsware.de> wrote:
>
>

Hi Bernd!

Thanks for the new patch version.

There are still some problems with tests.
0. You rely on order of test execution. "init.sql" test must prepend any other test. I doubt it is guaranteed.
1. There's a typo float8__buffering in Makefile
2. cidr type seems to be left behind
3. Tests do not seem to work when your configuration lacks injection points.

You can see how tests with injection points are excluded in other modules...

Perhaps, let's ask Michael.
Michael, we have 30 tests with checks that need injection points. But these 30 tests also test functionality that needs
tobe tested even in build without injection points. 
Do we have to extract checks with injection point into separate regression test? So that we can exclude this test in
buildswithout injection points. 

Thanks!


Best regards, Andrey Borodin.


pgsql-hackers by date:

Previous
From: Corey Huinker
Date:
Subject: Re: More CppAsString2() in psql's describe.c
Next
From: Dmitry Dolgov
Date:
Subject: Re: Changing shared_buffers without restart