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

[Andrew Gilligan]

On 20 Jan 2008, at 04:34, Tom Lane wrote:

Andrew Gilligan <[EMAIL PROTECTED]> 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

patch-83rc2-src-bin-psql-describe.c.diff
Description: Binary data

---------------------------(end of broadcast)---------------------------
TIP 7: You can help support the PostgreSQL project by donating at

                http://www.postgresql.org/about/donate