Re: Amcheck verification of GiST and GIN

From: Andrey Borodin <x4mmm(at)yandex-team(dot)ru>
To: Nikolay Samokhvalov <samokhvalov(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Amcheck verification of GiST and GIN
Date: 2022-06-25 19:10:11
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

> On 23 Jun 2022, at 00:27, Nikolay Samokhvalov <samokhvalov(at)gmail(dot)com> wrote:
> Since you're talking to yourself, just wanted to support you – this is an important thing, definitely should be very useful for many projects; I hope to find time to test it in the next few days.

Thanks Nikolay!

> On 23 Jun 2022, at 04:29, Andres Freund <andres(at)anarazel(dot)de> wrote:
Thanks for looking into the patch, Andres!

> On 2022-06-22 20:40:56 +0300, Andrey Borodin wrote:
>>> diff --git a/contrib/amcheck/amcheck.c b/contrib/amcheck/amcheck.c
>> new file mode 100644
>> index 0000000000..7a222719dd
>> --- /dev/null
>> +++ b/contrib/amcheck/amcheck.c
>> @@ -0,0 +1,187 @@
>> +/*-------------------------------------------------------------------------
>> + *
>> + * amcheck.c
>> + * Utility functions common to all access methods.
> This'd likely be easier to read if the reorganization were split into its own
> commit.
> I'd also split gin / gist support. It's a large enough patch that that imo
> makes reviewing easier.
I will split the patch in 3 steps:
1. extract generic functions to amcheck.c
2. add gist functions
3. add gin functions
But each this step is just adding few independent files + some lines to Makefile.

I'll fix other notes too in the next version.


Best regards, Andrey Borodin.

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Hannu Krosing 2022-06-25 20:54:39 Re: Hardening PostgreSQL via (optional) ban on local file system access
Previous Message Andrey Borodin 2022-06-25 17:17:28 Re: Hardening PostgreSQL via (optional) ban on local file system access