Re: BUG #13741: vacuumdb does not accept valid password

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>, brown(at)fastmail(dot)com, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: BUG #13741: vacuumdb does not accept valid password
Date: 2015-11-06 06:53:47
Message-ID: CAB7nPqTq5PfQbMMCqJ4KRM8Q1ppOgVu4WziuEzm=3TNUN5wUoA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Wed, Nov 4, 2015 at 4:30 PM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com> wrote:
> On Wed, Nov 4, 2015 at 11:16 AM, Alvaro Herrera
> <alvherre(at)2ndquadrant(dot)com> wrote:
>> Haribabu Kommi wrote:
>>> On Wed, Nov 4, 2015 at 11:24 AM, Haribabu Kommi
>>> <kommi(dot)haribabu(at)gmail(dot)com> wrote:
>>
>>> And one more thing, the vacuumdb password behavior is present in back branches
>>> also, is it worth back patching the vacuumdb fix to all supported
>>> branches and apply
>>> the jobs connection fix only to master and 9.5?
>>
>> Given the lack of complaints, I doubt it's worth the destabilization
>> risk. Let's just patch 9.5 and be done with it.
>
> Fine for me. --all is supported for ages.

OK, so attached is a patch aimed at master and 9.5. I reused the
suggestion of Haribabu to not rely completely on the maintenance
database after testing with a couple of database, some of them using
md5 and others trust. That's just more portable this way, and user
just needs to specify the password once to be done even with vacuumdb
--all.
Thoughts?
--
Michael

Attachment Content-Type Size
20151106_scripts_parallel_v3.patch text/x-patch 12.2 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Michael Paquier 2015-11-06 07:47:25 Re: Re: BUG #13685: Archiving while idle every archive_timeout with wal_level hot_standby
Previous Message Tom Lane 2015-11-05 23:18:45 Re: BUG #13756: jsonb_path_ops gin index produce empty result on nested array

Browse pgsql-hackers by date

  From Date Subject
Next Message Kouhei Kaigai 2015-11-06 07:02:48 Re: CustomScan support on readfuncs.c
Previous Message Michael Paquier 2015-11-06 06:12:12 Re: Beta2 Next week