On 2020-09-07 12:02, Asif Rehman wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, passed
> Implements feature: tested, passed
> Spec compliant: tested, passed
> Documentation: tested, passed
>
> The patch applies cleanly and looks fine to me. However wouldn't it be better to just map the CURRENT_ROLE to
CURRENT_USERin backend grammar?
Existing code treats them differently. I think, given that the code is
already written, it is good to preserve what the user wrote.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services