From: Vik Fearing <vik@postgresfriends.org>
To: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: Allow CURRENT_ROLE in GRANTED BY
Date: Wed, 24 Jun 2020 10:12:38 +0200
Message-ID: <f3723966-022a-fb74-06f4-3801f4fb7c1b@postgresfriends.org> (raw)
In-Reply-To: <f2feac44-b4c5-f38f-3699-2851d6a76dc9@2ndquadrant.com>
References: <f2feac44-b4c5-f38f-3699-2851d6a76dc9@2ndquadrant.com>
On 6/24/20 8:35 AM, Peter Eisentraut wrote:
> I was checking some loose ends in SQL conformance, when I noticed: We
> support GRANT role ... GRANTED BY CURRENT_USER, but we don't support
> CURRENT_ROLE in that place, even though in PostgreSQL they are
> equivalent. Here is a trivial patch to add that.
The only thing that isn't dead-obvious about this patch is the commit
message says "[PATCH 1/2]". What is in the other part?
Assuming that's just a remnant of development, this LGTM.
--
Vik Fearing
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: pgsql-hackers@postgresql.org
Cc: vik@postgresfriends.org, peter.eisentraut@2ndquadrant.com
Subject: Re: Allow CURRENT_ROLE in GRANTED BY
In-Reply-To: <f3723966-022a-fb74-06f4-3801f4fb7c1b@postgresfriends.org>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox