Re: [PATCH] Add size/acl information when listing databases

This has been saved for the 8.4 release:

http://momjian.postgresql.org/cgi-bin/pgpatches_hold

---------------------------------------------------------------------------

Andrew Gilligan wrote:
>
> On 20 Jan 2008, at 04:34, Tom Lane wrote:
> > Andrew Gilligan <andy(at)tcpd(dot)net> writes:
> >> Showing the size on \l+ probably makes more sense, but I imagine that
> >> would require a very different approach due to the permissions
> >> changes?
> >
> > I haven't experimented, but perhaps something along the lines of
> >
> > case when has_database_privilege(current_user, db.oid, 'connect')
> > then pg_database_size...
> > else null
> > end
> >
> > would fix it.
>
>
> Yep, that seems to do the trick.
>
> I've attached an updated patch (based on 8.3RC2) that adds the ACL
> information to \l and the size to \l+ if available.
>
> Best regards,
> -Andy
>
>

[ Attachment, skipping... ]

>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: You can help support the PostgreSQL project by donating at
>
> http://www.postgresql.org/about/donate

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +