From: | Israel Barth Rubio <barthisrael(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | vignesh C <vignesh21(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Add -k/--link option to pg_combinebackup |
Date: | 2025-03-03 14:00:29 |
Message-ID: | CAO_rXXDgmUhq_WPSUZjsoUaJtSncaTRXscDy_9yzLiiyebSiFw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Thanks for the new round of reviews!
> 1) The new file 010_links.pl added should be included to meson.build also
Added 010_links.pl to src/bin/pg_combinebackup/meson.build.
> 2) Since it is a new file, "Copyright (c) 2021-2025" should be
> "Copyright (c) 2025"
Done!
> 3) Since it is a single statement, no need of braces here
I missed removing the braces along with the warning hint in
the previous version. Adjusted.
> Also give it a different number than any existing file -- if we
> already have an 010 in that directory then make this one something
> else. 012, 050, or whatever makes most sense.
Oh, t/010_links.pl was not conflicting with anything.
The snippet that Vignesh quoted in his reply was from the
meson.build file of pg_basebackup instead of from
pg_combinebackup.
Attaching the new version of the patch in this reply.
I had to slightly change 010_links.pl and copy_file.c to
properly handle Windows, as the meson tests on
Windows had complaints with the code of v3:
* copy_file.c was forcing CopyFile on Windows regardless
of the option chosen by the user. Now, to make that behavior
backward compatible, I'm only forcing CopyFile on Windows
when the copy method is not --link. That allows my patch to
work, and doesn't change the previous behavior.
* Had to normalize paths when performing string matching in
the test that verifies the hard link count on files.
Best regards,
Israel.
Attachment | Content-Type | Size |
---|---|---|
v4-0001-pg_combinebackup-add-support-for-hard-links.patch | application/octet-stream | 17.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bertrand Drouvot | 2025-03-03 14:07:12 | Re: Log connection establishment timings |
Previous Message | Gabriele Bartolini | 2025-03-03 13:39:47 | Re: RFC: Additional Directory for Extensions |