Re: [patch] Add schema total size to psql \dn+

From: Julien Rouhaud <rjuju123(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Gilles Darold <gilles(dot)darold(at)dalibo(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [patch] Add schema total size to psql \dn+
Date: 2019-02-23 19:54:22
Message-ID: CAOBaU_Zb0n1NfK7xPeJ=RGvtziVNBMvqZMxXOKsRCoWSr5wkyg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Feb 22, 2019 at 7:22 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Is there any permissions issue involved here? I'd be a bit worried
> about whether \dn+ could fail, or deliver misleading answers, when
> run by a user without permissions on (some) tables. Also, even if
> we allow people to get size info on tables they can't read today,
> having this feature would be a roadblock to tightening that in
> the future.

Gilles' patch is using pg_total_relation_size(), so there's no
permission check at all. Also AFAICS this function even allows any
user to get the size of any other user backend's temporary table.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2019-02-23 21:27:44 Re: FETCH FIRST clause PERCENT option
Previous Message Chapman Flack 2019-02-23 19:23:17 Re: proposal: variadic argument support for least, greatest function