Re: Server instrumentation patch - Mailing list pgsql-hackers

From Michael Paesold
Subject Re: Server instrumentation patch
Date
Msg-id 034301c578dc$81cb26c0$0f01a8c0@zaphod
Whole thread Raw
In response to Re: Server instrumentation patch  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-hackers
Andreas Pflug wrote:

> Michael Paesold wrote:
>
>> Andreas Pflug wrote:
>>
>>>> For the second, please supply a patch that moves _all_ of dbsize into
>>>> the main server.  I think we have agreement on that.
>>>>
>>>
>>> I don't think so. As I mentioned, those views are broken. Do you want 
>>> them to be in core anyway?
>>
>>
>> Why is e.g. this one broken:
>> int8 database_size(name) - Return the size of the database in
>> bytes (by name)
>>
>> It should do the same as the one with the oid except that it will resolve 
>> the name first, no? If not it should be fixed, not dropped. I understand 
>> you'd like to have those functions for the GUI frontends, but what about 
>> psql users? For many people it will be hard work to type the subquery to 
>> get the database oid.
>>
>> I vote for all (possibly corrected) functions to be moved into core.
>
> You're correct about the functions, but I was talking about the views. 
> "WHERE name = $1" won't respect the schema, contrary to the current doc.

Oh, I am sorry for not reading carefully enough. Didn't know there were 
views at all. So if they are broken and cannot be fixed, well...

Best Regards,
Michael Paesold 



pgsql-hackers by date:

Previous
From: Josh Berkus
Date:
Subject: Re: [PATCHES] O_DIRECT for WAL writes
Next
From: "Michael Paesold"
Date:
Subject: Re: Server instrumentation patch