Re: review: ALTER ROLE ALL SET

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: review: ALTER ROLE ALL SET
Date: 2013-02-18 04:50:09
Message-ID: 1361163009.6225.0.camel@vanquo.pezone.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, 2013-02-13 at 08:38 +0100, Pavel Stehule wrote:
> 5) Open question
>
> * I think so doc is not fully correct
>
> http://www.postgresql.org/message-id/CAFj8pRBf6suKewDCiXiGy=NeYY_Ns9CAZemomvRYsAQ=UpLzNA@mail.gmail.com

Fixed that and committed.

> * syntax
>
> when I try some variants I got not user friendly error message
>
> postgres=# alter role set work_mem = '1MB';
> ERROR: unrecognized role option "work_mem"
> LINE 1: alter role set work_mem = '1MB';
> ^
>
This issue already existed before. I don't think we can do much about
it with one token lookahead.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Heikki Linnakangas 2013-02-18 08:11:08 Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq
Previous Message Amit Kapila 2013-02-18 04:07:12 Re: [PATCH] Add PQconninfoParseParams and PQconninfodefaultsMerge to libpq