From: | Jaime Casanova <jaime(at)2ndquadrant(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Greg Smith <greg(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Review of pg_archivecleanup -x option patch |
Date: | 2012-03-09 05:46:42 |
Message-ID: | CAJKUy5i8Otnb7QvT0HwjaWt5-aUbrZXqh+HD4z1caud706Do_Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Mar 8, 2012 at 8:25 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Sun, Feb 12, 2012 at 9:19 PM, Greg Smith <greg(at)2ndquadrant(dot)com> wrote:
>> The smaller step of automatically stripping the specified suffix from the
>> input file name, when it matches the one we've told the program to expect,
>> seems like a usability improvement unlikely to lead to bad things though.
>> I've certainly made the mistake you've shown when using the patched version
>> of the program myself, just didn't think about catching and correcting it
>> myself before. We can rev this to add that feature and resubmit easily
>> enough, will turn that around soon.
>
> This change seems plenty small enough to slip in even at this late
> date, but I guess we're lacking a new version of the patch.
>
Sorry, here's the patch rebased and with the suggestion from Alex.
Which reminds me, I never thank him for the review (shame on me) :D
--
Jaime Casanova www.2ndQuadrant.com
Professional PostgreSQL: Soporte 24x7 y capacitación
From | Date | Subject | |
---|---|---|---|
Next Message | Jaime Casanova | 2012-03-09 05:47:14 | Re: Review of pg_archivecleanup -x option patch |
Previous Message | Tom Lane | 2012-03-09 05:00:01 | Re: pgsql_fdw, FDW for PostgreSQL server |