Horton

Thread

Commits

  1. Make has_sequence_privilege support WITH GRANT OPTION

  1. has_sequence_privilege() never got the memo

    Joe Conway <mail@joeconway.com> — 2017-11-23T02:01:38Z

    I just noticed that has_sequence_privilege() never got the memo about
"WITH GRANT OPTION". Any objections to the attached going back to all
supported versions?

Joe

-- 
Crunchy Data - http://crunchydata.com
PostgreSQL Support for Secure Enterprises
Consulting, Training, & Open Source Development

  2. Re: has_sequence_privilege() never got the memo

    Tom Lane <tgl@sss.pgh.pa.us> — 2017-11-23T03:58:05Z

    Joe Conway <mail@joeconway.com> writes:
> I just noticed that has_sequence_privilege() never got the memo about
> "WITH GRANT OPTION". Any objections to the attached going back to all
> supported versions?

That looks odd.  Patch certainly makes this case consistent with the
rest of acl.c, but maybe there's some deeper reason?  Peter?

			regards, tom lane

  3. Re: has_sequence_privilege() never got the memo

    Peter Eisentraut <peter.eisentraut@2ndquadrant.com> — 2017-11-23T15:16:35Z

    On 11/22/17 22:58, Tom Lane wrote:
> Joe Conway <mail@joeconway.com> writes:
>> I just noticed that has_sequence_privilege() never got the memo about
>> "WITH GRANT OPTION". Any objections to the attached going back to all
>> supported versions?
> 
> That looks odd.  Patch certainly makes this case consistent with the
> rest of acl.c, but maybe there's some deeper reason?  Peter?

No I think it was just forgotten.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

  4. Re: has_sequence_privilege() never got the memo

    Joe Conway <mail@joeconway.com> — 2017-11-26T17:53:14Z

    On 11/23/2017 07:16 AM, Peter Eisentraut wrote:
> On 11/22/17 22:58, Tom Lane wrote:
>> Joe Conway <mail@joeconway.com> writes:
>>> I just noticed that has_sequence_privilege() never got the memo about
>>> "WITH GRANT OPTION". Any objections to the attached going back to all
>>> supported versions?
>> 
>> That looks odd.  Patch certainly makes this case consistent with the
>> rest of acl.c, but maybe there's some deeper reason?  Peter?
> 
> No I think it was just forgotten.

Pushed.

Joe

-- 
Crunchy Data - http://crunchydata.com
PostgreSQL Support for Secure Enterprises
Consulting, Training, & Open Source Development