Re: Tablespace patch review - Mailing list pgsql-patches

From Tom Lane
Subject Re: Tablespace patch review
Date
Msg-id 25205.1087590190@sss.pgh.pa.us
Whole thread Raw
In response to Re: Tablespace patch review  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: Tablespace patch review  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Tom Lane wrote:
>> Somebody's got to fix oid2name and dbsize though.  Bruce, you want
>> to catch those?

> Uh, how do they have to be fixed?  Isn't the relfilenode unchanged?  Do
> we just need to add tablespace lookups?

How useful will oid2name be if it doesn't understand about tablespaces?
I dunno how it ought to be changed, but surely it needs some thought.

dbsize doesn't even compile right now, because it's using
GetDatabasePath which now has another argument.  I did not patch it
because it needs more thought: should it report the total of all
tablespaces for the database, or should its API be extended so you
can ask about individual tablespaces, or what?  In any case it's
not a one-liner fix...

            regards, tom lane

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Tablespace patch review
Next
From: Gavin Sherry
Date:
Subject: Re: Tablespace patch review