* Tom Lane (t...@sss.pgh.pa.us) wrote: > Robert Haas <robertmh...@gmail.com> writes: > > Ugg, wait a minute. This not only adds %U; it also changes the > > behavior of %u, which I don't think we've agreed on. Also, emitting > > 'none' when not SET ROLE has been done is pretty ugly. I'm back to > > thinking we need to push this out to 9.2 and take more time to think > > about this. > > Yeah, I thought what was supposed to be emitted was the value of > current_user, not SQL's weird definition of what SET ROLE means.
current_user uses GetUserNameFromId() and goes through the cache lookups
to get there. I was using what show_role() returns (which is also what
'show role;' returns). I'd be happy to make it emit an empty string
when 'none' is returned though.
Thanks,
Stephen
signature.asc
Description: Digital signature
