From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Pierre Ducroquet <p(dot)psql(at)pinaraf(dot)info> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Ryan Murphy <ryanfmurphy(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Small patch for pg_basebackup argument parsing |
Date: | 2017-10-02 00:36:49 |
Message-ID: | 64C3167E-60D0-471F-9DA6-4B14D935C8ED@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 18 Sep 2017, at 23:18, Pierre Ducroquet <p(dot)psql(at)pinaraf(dot)info> wrote:
>
> On Monday, September 18, 2017 5:13:38 PM CEST Tom Lane wrote:
>> Ryan Murphy <ryanfmurphy(at)gmail(dot)com> writes:
>>> 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!
>>
>> I took a quick look through this and had a few thoughts:
>
> I agree with your suggestions, I will try to produce a new patch before the
> end of the week.
This patch has been marked as Returned with Feedback as no new version has been
submitted. Please re-submit the new version of this patch to the next commit-
fest when it’s ready.
cheers ./daniel
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2017-10-02 00:43:41 | Re: [WIP] Zipfian distribution in pgbench |
Previous Message | Daniel Gustafsson | 2017-10-02 00:33:12 | Re: Optional message to user when terminating/cancelling backend |