From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Marco Nenciarini <marco(dot)nenciarini(at)2ndquadrant(dot)it> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Updated backup APIs for non-exclusive backups |
Date: | 2016-03-19 12:15:02 |
Message-ID: | CABUevExmvJj1vxV8+SSnSbjmbOLkgrBEq_ro=okPuq9nAb++9w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Mar 2, 2016 at 6:49 PM, Marco Nenciarini <
marco(dot)nenciarini(at)2ndquadrant(dot)it> wrote:
> Hi Magnus,
>
Hi!
First, again my apologies for completely missing that you had posted this
review!
> I've finally found some time to take a look to the patch.
>
> It applies with some fuzziness on master, but the result looks correct.
> Unfortunately the OID of the new pg_stop_backup function conflicts with
> "pg_blocking_pids()" patch (52f5d578d6c29bf254e93c69043b817d4047ca67).
>
Fixed, thanks!
> After changing it the patch does not compile:
>
It compiles fine for me, and with no warnings.
>
> gcc -Wall -Wmissing-prototypes -Wpointer-arith
> -Wdeclaration-after-statement -Wendif-labels -Wmissing-format-attribute
> -Wformat-security -fno-strict-aliasing -fwrapv
> -Wno-unused-command-line-argument -g -O0 -g -fno-omit-frame-pointer
> -I../../../../src/include
>
> -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/libxml2
> -I/usr/local/include -I/usr/local/opt/openssl/include -c -o xlog.o
> xlog.c -MMD -MP -MF .deps/xlog.Po
> xlog.c:10000:19: error: use of undeclared identifier 'tblspc_mapfbuf';
>
Eh. There is no presence of "tblspc_mapfbuf" after the patch. I think it
looks like the "applies with fuzziness" actually wasn't correct, and you
ended up with bad code with a mix of the old and the new code in it.
I've attached an updated patch, which is rebased on current master and
includes the oid fix.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
Attachment | Content-Type | Size |
---|---|---|
backup_api2.patch | text/x-patch | 19.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dilip Kumar | 2016-03-19 12:22:27 | Re: Move PinBuffer and UnpinBuffer to atomics |
Previous Message | Amit Kapila | 2016-03-19 11:14:49 | Re: Performance degradation in commit ac1d794 |