> Maybe keeping track of 'role' via ParameterStatus messages is a good
> idea for reasons unrelated to this patch -- maybe it can be useful for
> applications to be aware of role changes -- but I'm not 100% sure about
> that, and in particular I'm not sure how heavy the protocol traffic is
> going to be if such messages are emitted every time you run a security
> invoker function or things like that
With the latest version of the patch, 'role' is not needed as
'session authorization' is not shown either [1].
The latest version LGTM.
Regards,
Sami Imseih
Amazon Web Services (AWS)
[1] https://www.postgresql.org/message-id/CAA5RZ0tbWopM83akPZ5M42V_RtyMTV8UfNUdE9LYw0YsPdOX5g%40mail.gmail.com
