Re: [HACKERS] For review: Server instrumentation patch

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Andreas Pflug <pgadmin(at)pse-consulting(dot)de>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>, Dave Page <dpage(at)vale-housing(dot)co(dot)uk>
Subject: Re: [HACKERS] For review: Server instrumentation patch
Date: 2005-08-12 12:40:59
Message-ID: 20050812124059.GA12111@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Fri, Aug 12, 2005 at 11:28:22AM +0000, Andreas Pflug wrote:
> Bruce Momjian wrote:
>
> >
> >I did not include pg_logdir_ls because that was basically pg_ls_dir with
> >logic to decode the log file name and convert it to a timestamp. That
> >seemed best done in the client.
>
> IMHO omitting pg_logdir_ls is a bad idea, because the function is
> intended to hide server internal's naming scheme from the user. We want
> as few server side implementation specific client side code as possible.

BTW, it surprised me that one of the functions (don't remember which
one) expected the log files to be named in a very specific fashion. So
there's no flexibility for changing the log_prefix. Probably it's not
so bad, but strange anyway. Is this for "security" reasons?

--
Alvaro Herrera (<alvherre[a]alvh.no-ip.org>)
Y dijo Dios: "Que sea Satanás, para que la gente no me culpe de todo a mí."
"Y que hayan abogados, para que la gente no culpe de todo a Satanás"

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-08-12 15:37:51 Re: Bug in canonicalize_path()
Previous Message Andreas Pflug 2005-08-12 11:28:22 Re: [HACKERS] For review: Server instrumentation patch