On Wed, Mar 8, 2017 at 2:32 PM, Rushabh Lathia <rushabh.lat...@gmail.com> wrote:
> One quick comments:
>
> +            | PASSWORD '(' Sconst USING Sconst ')'
> +                {
> +                    $$ = makeDefElem("methodPassword",
> +                                     (Node *)list_make2(makeString($3),
> +                                                        makeString($5)),
> +                                     @1);
> +                }
>
> methodPassword looks bit weird, can we change it to passwordMethod
> or pwdEncryptMethod ?

Using "Password" in suffix looks still better to me though for
consistency with the other ones.
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to