From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Dave Page <dpage(at)vale-housing(dot)co(dot)uk> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: For review: Server instrumentation patch |
Date: | 2005-07-23 19:20:23 |
Message-ID: | 200507231920.j6NJKNk28858@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Dave Page wrote:
>
>
> > -----Original Message-----
> > From: Bruce Momjian [mailto:pgman(at)candle(dot)pha(dot)pa(dot)us]
> > Sent: 23 July 2005 20:01
> > To: Dave Page
> > Cc: PostgreSQL-development
> > Subject: Re: [HACKERS] For review: Server instrumentation patch
> >
> >
> > This patch looks good. The only question I have is why you
> > didn't want
> > the pgport rename/unlink calls? We usually use them unless there is
> > some reason not to.
>
> <thinks...> Probably because this was written originally for 7.4 (as a
> pgAdmin contrib module) and I'm guessing the pgport rename/unlink
> weren't there at that time. I can't think of any reason not to use them
> - do you want an updated patch or are you OK to tweak it when applying?
No, I modified my version. Thanks.
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Alexey Borzov | 2005-07-23 19:47:09 | Re: [HACKERS] Enticing interns to PostgreSQL |
Previous Message | Dave Page | 2005-07-23 19:11:26 | Re: For review: Server instrumentation patch |