Re: [PATCHES] Dbsize backend integration - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [PATCHES] Dbsize backend integration
Date
Msg-id 200507060118.j661I1T00525@candle.pha.pa.us
Whole thread Raw
In response to Re: [PATCHES] Dbsize backend integration  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCHES] Dbsize backend integration  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane wrote:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > If we go pg_table_size() and pg_relation_size(), which is object-only
> > and which is heap + index + toast?  I think ideally we want
> > pg_relation_size to be the combined one, but then we have pg_table_size
> > that works on indexes and toast too, and that is confusing, and we don't
> > want to add index and toast versions.  Or is an index a relation?  And
> > TOAST?
>
> All the backend code thinks so --- anything that has an entry in
> pg_class is a relation.  So personally I don't find "table" and
> "relation" confusing in this context.  But I can see it might be
> confusing to people not familiar with PG jargon.
>
> > OK, how about pg_relation_size for heap/index/toast, and
> > pg_complete_relation_size for the combined total.
>
> I could live with that.  Or "pg_total_relation_size".

The problem with "total", to me, is that it already is the total size of
the heap/index/toast.  Complete has the idea of adding additional
pieces, which I think fits best.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

pgsql-hackers by date:

Previous
From: Andrew - Supernews
Date:
Subject: oids vs composite types, in cvs head
Next
From: Andrew Dunstan
Date:
Subject: Re: timezone changes break windows and cygwin