From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>, David Steele <david(at)pgmasters(dot)net> |
Subject: | Re: Support for pg_receivexlog --format=plain|tar |
Date: | 2016-12-28 23:35:37 |
Message-ID: | CAB7nPqSnewDeixx+djcfSRT6AhjB8fRPDwtCe2xAzcz_FvbMiw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Dec 28, 2016 at 9:31 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> Conditional tests? It probably wouldn't hurt to have them, but that would be
> something more generic (like we'd need something to actually validate it --
> but it would make sense to have a test that, with compression enabled, would
> verify if the uncompressed file turns out to be exactly 16Mb for example).
Looking at if the build is compiled with libz via SQL or using
pg_config is the way to go here. A minimum is doable.
>> A couple of things to be aware of though:
>> - gzopen, gzwrite and gzclose are used to handle the gz files. That's
>> unconsistent with the tar method that is based on mainly deflate() and
>> more low level routines.
>
> But chosen for simplicity, I assume?
Yep. That's quite in-line with the current code.
>> - I have switched the directory method to use a file pointer instead
>> of a file descriptor as gzwrite returns int as the number of
>> uncompressed bytes written.
>
> I don't really follow that reasoning :) Why does the directory method have
> to change to use a filepointer because of that?
The only reason is that write() returns size_t and fwrite returns int,
while gzwrite() returns int. It seems more consistent to use fwrite()
in this case. Or we don't bother about my nitpicking and just cast
stuff.
>> What do you think about this approach? I'll add that to the next CF.
>
> I haven't reviweed the code in detail but yes, I think this approach is the
> right one.
OK, thanks. I'll think about those conditional tests, correct one or
two things in the patch and submit again soon.
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2016-12-29 03:33:50 | Re: pg_stat_activity.waiting_start |
Previous Message | Alvaro Herrera | 2016-12-28 23:20:18 | rewrite HeapSatisfiesHOTAndKey |