Skip site navigation (1) Skip section navigation (2)

Re: Tablespace patch review

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Gavin Sherry <swm(at)linuxworld(dot)com(dot)au>,pgsql-patches(at)postgresql(dot)org
Subject: Re: Tablespace patch review
Date: 2004-06-18 20:23:10
Message-ID: 25205.1087590190@sss.pgh.pa.us (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-patches
Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)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

In response to

Responses

pgsql-hackers by date

Next:From: Carlos Guzmán ÁlvarezDate: 2004-06-18 20:23:57
Subject: Re: Timestamp format question
Previous:From: Bruce MomjianDate: 2004-06-18 20:20:13
Subject: Re: [PATCHES] Configuration patch

pgsql-patches by date

Next:From: Gavin SherryDate: 2004-06-18 21:42:04
Subject: Re: Tablespace patch review
Previous:From: Bruce MomjianDate: 2004-06-18 20:20:13
Subject: Re: [PATCHES] Configuration patch

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group