Thread: has_sequence_privilege() never got the memo

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

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

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

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