> On 23 Jun 2022, at 00:27, Nikolay Samokhvalov <samokhva...@gmail.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 <and...@anarazel.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.
Thanks!
Best regards, Andrey Borodin.
