public inbox for [email protected]
help / color / mirror / Atom feedFrom: Michael Paquier <[email protected]>
To: David G. Johnston <[email protected]>
Cc: [email protected] <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: CREATE/ALTER ROLE with NULL password
Date: Fri, 23 Nov 2018 09:13:26 +0900
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAKFQuwZTPLjY-Be+YUjMaKjMVr8pino8Q3n9iwUH5QN_Qee3Tw@mail.gmail.com>
References: <[email protected]>
<[email protected]>
<CAKFQuwbugMRHCTDiZ-1G=9cHs06c6pv_dS44x4GLQ=2q5ZV2yw@mail.gmail.com>
<[email protected]>
<CAKFQuwZTPLjY-Be+YUjMaKjMVr8pino8Q3n9iwUH5QN_Qee3Tw@mail.gmail.com>
On Thu, Nov 22, 2018 at 09:54:07AM -0700, David G. Johnston wrote:
> On Thursday, November 22, 2018, Michael Paquier <[email protected]> wrote:
>> The docs list the following with the patch as supported grammar:
>> [ ENCRYPTED ] PASSWORD 'password' | PASSWORD NULL
>> And it seems to me that '|' has priority over '[]', so ENCRYPTED does
>> not apply to PASSWORD NULL if phrased this way.
>
> Yes, the syntax block is perfectly clear but we still explain said grammer
> in words and should be precise there as well, IMO. Not a big deal though.
Okay, thanks. I have committed the simplest version.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
view thread (6+ messages)
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: CREATE/ALTER ROLE with NULL password
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox