Re: [PATCH] Add tests for Bitmapset - Mailing list pgsql-hackers

From Greg Burd
Subject Re: [PATCH] Add tests for Bitmapset
Date
Msg-id CA0C373A-6857-4C4C-BF67-5C72AABA0E0D@greg.burd.me
Whole thread Raw
In response to Re: [PATCH] Add tests for Bitmapset  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: [PATCH] Add tests for Bitmapset
List pgsql-hackers
On Sep 16 2025, at 8:02 am, Robert Haas <robertmhaas@gmail.com> wrote:

> On Tue, Sep 16, 2025 at 2:04 AM Michael Paquier <michael@paquier.xyz> wrote:
>> one SQL function mapping to each C function we are testing?
>
> Yes, I think we should do this, if possible.

Michael, Robert,

Thanks for your time reviewing the proposed code and for providing
feedback. This patch started life as simply copy of the test_radixtree
module which uses a single test function triggered in SQL, hence my
approach.  I guess I should have copied the test_slru module instead! :)

I see the value in the idea of splitting it up into separate functions
and I'll give that a try.

Michael, apologies for the white space issues.  I'll clean that up, and
thank you for the `git diff --check` tip, I'll add that to my toolbox/routine.

> --
> Robert Haas
> EDB: http://www.enterprisedb.com

best.

-greg




pgsql-hackers by date:

Previous
From: "Core Studios Inc."
Date:
Subject: Re: Incorrect result of bitmap heap scan.
Next
From: Matthias van de Meent
Date:
Subject: Re: Incorrect result of bitmap heap scan.