SEO

#1 (**permalink**) 04-11-2008, 04:33 AM

> -----Original Message-----
> From: Michael Paesold [mailto:mpaesold@gmx.at]
> Sent: 24 June 2005 17:53
> To: Dave Page; Andreas Pflug
> Cc: PostgreSQL-development
> Subject: Re: [HACKERS] Server instrumentation patch
>
> > My main concern is that the names are inconsistent for no obvious
> > reason.
>
> That could be fixed by having:
> pg_database_size(name)
> pg_database_size(oid)
>
> The original idea was probably to name "internal" functions
> with pg_ and
> more user friendly ones without pg_. That does not mean it's
> a good idea.

Yes, agreed - it could be fixed that way easily. If the inclusion of
/all/ functions is for backwards compatibility though, then that change
is somewhat more of a problem.

>
> Well, I don't feel this is really bloat. I have been using
> them since the
> creation of the contrib module and have found them quite useful.

Fair enough.

Regards, Dave.

---------------------------(end of broadcast)---------------------------
TIP 5: Have you checked our extensive FAQ?

http://www.postgresql.org/docs/faq

#2 (**permalink**) 04-11-2008, 04:33 AM

Dave Page wrote:
>
>
> > -----Original Message-----
> > From: Michael Paesold [mailto:mpaesold@gmx.at]
> > Sent: 24 June 2005 17:53
> > To: Dave Page; Andreas Pflug
> > Cc: PostgreSQL-development
> > Subject: Re: [HACKERS] Server instrumentation patch
> >
> > > My main concern is that the names are inconsistent for no obvious
> > > reason.
> >
> > That could be fixed by having:
> > pg_database_size(name)
> > pg_database_size(oid)
> >
> > The original idea was probably to name "internal" functions
> > with pg_ and
> > more user friendly ones without pg_. That does not mean it's
> > a good idea.
>
> Yes, agreed - it could be fixed that way easily. If the inclusion of
> /all/ functions is for backwards compatibility though, then that change
> is somewhat more of a problem.

We are moving the functions into the backend so if we are going to make
them more consistent, now is the time. People are already going to not
have to load them from /contrib, so a more consistent API change is fine
at this stage --- it will be much harder later.

--
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

---------------------------(end of broadcast)---------------------------
TIP 8: explain analyze is your friend

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

SEO

Re: Server instrumentation patch