Hi Ramanarayana,
Thank you for your review and suggestion. Please see the attached updated patch.
> Issues found while testing
>1) If invalid value is given to PGLOGMINLEVEL, empty log file is created which should not happen.
Thank you for your test. However in my dev environment, empty log file is not created.
Could you explain more detail about 1)? I will check it again.
>2) If log file size exceeds the value configured in PGLOGSIZE, new log file is not getting created.
About 2) (and may be 1) ), perhaps is this something like that?
There are my mistake about first line information of created log file
"The maximum size of this log is %s *Bytes*, the parameter 'logminlevel' is set to %s\n".
- Maximum size is not bytes but megabytes.
- Display logminlevel which set by user. Internally, an invalid value is not set to logminlevel.
So trust the created log file first line info, if you set PGLOGSIZE=1000 as the meaning of "set maximum log size to
1000Bytes",
a new file was not created even if it exceeds 1000 bytes.
If it is correct, I fixed the comment to output internal setting log maximum size and user setting value.
And if you set PGLOGMINLEVEL to invalid value (ex. "aaa"), it is not set to the parameter; The default value (level1)
isset internally.
I fixed first line comment to output notification " if invalid value, level1(default) is set".
>3) If PGLOGSIZE is greater than 2048 bytes, log file is not created. Is this expected behavior?
Yes. I limit log file size.
>4) In the log file, an extra new line is present whenever the query is printed. Is this intentional?
Thank you, I fixed.
>5)Documentation for this feature is having grammatical errors and some spelling errors which can be looked into.
Thank you. I am checking my documentation now. I will fix it.
> Feedback in the code
Thank you. I fixed my code issue.
> Suggestions
I'll consider that...
Could you explain more about the idea?
Regards,
Aya Iwata
