Re: [HACKERS] Small patch for pg_basebackup argument parsing - Mailing list pgsql-hackers

From Ryan Murphy
Subject Re: [HACKERS] Small patch for pg_basebackup argument parsing
Date
Msg-id 20170918051107.1354.37488.pgcf@coridan.postgresql.org
Whole thread Raw
In response to Re: [HACKERS] Small patch for pg_basebackup argument parsing  (Ryan Murphy <ryanfmurphy@gmail.com>)
Responses Re: [HACKERS] Small patch for pg_basebackup argument parsing
List pgsql-hackers
The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           tested, passed
Documentation:            not tested

I applied this patch via patch -p1.  (Had an issue using git apply, but apparently that's common?)

All tests passed normally.  Ran make check, make installcheck, and make installcheck-world.

Looked thru the diffs and it looks fine to me.
Changing a lot of a integer/long arguments that were being read directly via atoi / atol
to be read as strings first, to give better error handling.

This looks good to go to me.  Reviewing this as "Ready for Committer".
Thanks for the patch, Pierre!

Ryan

The new status of this patch is: Ready for Committer

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

pgsql-hackers by date:

Previous
From: Amit Khandekar
Date:
Subject: Re: [HACKERS] Parallel Append implementation
Next
From: Thomas Munro
Date:
Subject: Re: [HACKERS] valgrind vs. shared typmod registry