Re: Review of pg_archivecleanup -x option patch

On Wed, Mar 28, 2012 at 12:13 PM, Jaime Casanova <jaime@2ndquadrant.com> wrote:
> On Wed, Mar 28, 2012 at 8:47 AM, Robert Haas <robertmhaas@gmail.com> wrote:
>>>>
>>>> $ ./pg_archivecleanup -x "bz2" /tmp 000000010000000100000058
>>>
>>> Hmm, but I thought that the idea was that the extension was optional.
>>> Perhaps I'm missing something but I don't think the previous patch
>>> will complain about that either; or at least I don't see why the
>>> behavior should be any different.
>>
>> Can someone enlighten me on this point?
>>
>
> mmm! you're right... it's not complaining either... i was sure it was...
> and i'm not sure i want to contor things for that...
>
> so, just forget my last mail about that... your refactor is just fine for me

OK, committed that way.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company