| From: | "Iwata, Aya" <iwata(dot)aya(at)jp(dot)fujitsu(dot)com> |
|---|---|
| To: | "Nagaura, Ryohei" <nagaura(dot)ryohei(at)jp(dot)fujitsu(dot)com>, 'Jacob Champion' <pchampion(at)pivotal(dot)io>, 'Jim Doty' <jdoty(at)pivotal(dot)io> |
| Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>, "nagata(at)sraoss(dot)co(dot)jp" <nagata(at)sraoss(dot)co(dot)jp>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, 'Haribabu Kommi' <kommi(dot)haribabu(at)gmail(dot)com>, 'Peter Eisentraut' <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
| Subject: | RE: libpq debug log |
| Date: | 2019-02-18 08:33:28 |
| Message-ID: | 71E660EB361DF14299875B198D4CE5423DED92AD@g01jpexmbkw25 |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
Because I mistook something about how to reply to e-mails,
my last reply is not reflected in the thread.
Response to Nagaura san's review point, I fixed all his review notes, except for pointing out about psql.
Please see the attached updated patch.
> 1)
> It would be better making the log format the same as the server log format,
I changed date style and added timezone.
> 2)
> It was difficult for me to understand the first line message in the log file.
I changed the message as "The maximum size of this log is 3 Bytes, the parameter 'logminlevel' is set to level2 ".
> 3)
> Under the circumstance that the environment variables "PGLOGDIR" and
> "PGLOGSIZE" are set correctly, the log file will also be created when the
> user connect the server with "psql".
> Does this follow the specification you have thought?
> Is there any option to unset only in that session when you want to connect
> with "psql"?
By separating session using Tera Term or screen command, you can do what you want.
So I didn't make the code complicated by implementing the option.
> 4)
> Your patch affects the behavior of PQtrace().
Thank you. I fixed.
Regards,
Aya Iwata
| Attachment | Content-Type | Size |
|---|---|---|
| v6-0001-libpq-trace-log.patch | application/octet-stream | 25.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jakub Glapa | 2019-02-18 09:11:10 | Re: dsa_allocate() faliure |
| Previous Message | Arthur Zakirov | 2019-02-18 08:23:46 | Re: [PATCH] xlogreader: do not read a file block twice |