RE: libpq debug log - Mailing list pgsql-hackers

From Iwata, Aya
Subject RE: libpq debug log
Date
Msg-id 71E660EB361DF14299875B198D4CE5423DED92AD@g01jpexmbkw25
Whole thread Raw
In response to RE: libpq debug log  ("Nagaura, Ryohei" <nagaura.ryohei@jp.fujitsu.com>)
Responses Re: libpq debug log  (Ramanarayana <raam.soft@gmail.com>)
RE: libpq debug log  ("Jamison, Kirk" <k.jamison@jp.fujitsu.com>)
List 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

pgsql-hackers by date:

Previous
From: Arthur Zakirov
Date:
Subject: Re: [PATCH] xlogreader: do not read a file block twice
Next
From: "Higuchi, Daisuke"
Date:
Subject: RE: [Bug Fix] ECPG: could not use some CREATE TABLE AS syntax