Re: Re: [HACKERS] Proposal: generic WAL compression

From: David Steele <david(at)pgmasters(dot)net>
To: Oleg Ivanov <o(dot)ivanov(at)postgrespro(dot)ru>
Cc: Stephen Frost <sfrost(at)snowman(dot)net>, Antonin Houska <ah(at)cybertec(dot)at>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Re: [HACKERS] Proposal: generic WAL compression
Date: 2018-02-07 15:31:09
Message-ID: f9841962-4aa3-1b67-007f-b6d73500a1d0@pgmasters.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Oleg,

On 1/22/18 4:37 PM, Stephen Frost wrote:
> Oleg,
>
> I'm not really sure why this is still in Needs Review as a review was
> posted and I don't see any follow-up. I've changed this to be Waiting
> for Author.
>
> * Antonin Houska (ah(at)cybertec(dot)at) wrote:
>> Oleg Ivanov <o(dot)ivanov(at)postgrespro(dot)ru> wrote:
>>
>>> In order to overcome that issue, I would like to propose the patch, which
>>> provides possibility to use another approach of the WAL record
>>> construction.
>>
>> After having spent several hours reviewing this patch I dare to send the
>> following comments:
>
> Thanks for the review Antonin!
>
>> * writeToPtr() and readFromPtr() are applied to the existing code. I think
>> this is a reason for a separate diff, to be applied before the actual patch.
>
> I'm not really a fan of using these, for my 2c. I'm not sure how others
> feel, but having these macros which change one of the arguments to the
> macro (by advancing the pointer) doesn't result in a net improvement to
> readability for me.
>
> The other review comments also seem pretty reasonable to me.

This proposal is still in need of review and hasn't really gotten it in
the last few CFs.

Since the feature does not seem like a good fit for the last CF of PG11
I am marking it Returned with Feedback.

Regards,
--
-David
david(at)pgmasters(dot)net

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David Steele 2018-02-07 15:39:49 Re: Re: New gist vacuum.
Previous Message Robert Haas 2018-02-07 15:25:04 Re: MCV lists for highly skewed distributions