| From: | Andreas Pflug <pgadmin(at)pse-consulting(dot)de> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | "Ed L(dot)" <pgsql(at)bluepolka(dot)net>, pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: log_filename_prefix --> log_filename + strftime() |
| Date: | 2004-08-27 19:08:51 |
| Message-ID: | 412F86C3.30304@pse-consulting.de |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
Tom Lane wrote:
> "Ed L." <pgsql(at)bluepolka(dot)net> writes:
>
>>Attached is a patch which replaces the 'log_filename_prefix' configuration
>>directive with a similar 'log_filename' directive.
>> + changes the default log filename to exclude the PID;
>
>
> This would be better stated as "makes it impossible to use the PID
> in the file name". While I'm prepared to grant that it may not be
> necessary to do so in many scenarios, I'm not very happy with
> arbitrarily removing the ability ... especially without giving any
> justification.
I don't have the time now to review the impact, but this might make
interpreting the log filename difficult or impossible, effectively
corrupting pg_logdir_ls.
I don't object against adjusting the timestamp format in a reasonable
way, but it should stay fixed; same about PID.
Regards,
Andreas
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2004-08-27 19:15:56 | Re: log_filename_prefix --> log_filename + strftime() |
| Previous Message | Ed L. | 2004-08-27 19:08:36 | Re: log_filename_prefix --> log_filename + strftime() |