| From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
|---|---|
| To: | "Ed L(dot)" <pgsql(at)bluepolka(dot)net> |
| Cc: | pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: log_line_prefix additions |
| Date: | 2004-08-25 22:25:49 |
| Message-ID: | 412D11ED.20401@dunslane.net |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
Ed L. wrote:
>Attached also is a patch to comments in sample postgresql.conf file.
>
>Subject: [PATCHES] log_line_prefix additions
>Date: Wednesday August 25 2004 3:26
>From: "Ed L." <pgsql(at)bluepolka(dot)net>
>To: pgsql-patches(at)postgresql(dot)org
>
>This patch against 8.0.0beta1 source adds log_line_prefix options for
>millisecond timestamps (%m), remote host (%h), and remote port (%P). The
>milliseconds are useful for QPS measurements, and the remote port is
>worthless to us as part of %r.
>
>
>
[snip]
>+ struct timezone tz = {0, 0};
>+
>+ gettimeofday(&tv, &tz);
>
>
>
The timezone is pointless. Just pass NULL as the second argument to
gettimeofday().
Also, I don't understand what you mean by the remote port being
"worthless to us as part of %r." Please explain.
cheers
andrew
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2004-08-25 22:44:33 | Re: add hint |
| Previous Message | James William Pye | 2004-08-25 22:24:16 | Re: add hint |