From: | "Karl O(dot) Pinc" <kop(at)meme(dot)com> |
---|---|
To: | Gilles Darold <gilles(dot)darold(at)dalibo(dot)com> |
Cc: | Christoph Berg <myon(at)debian(dot)org>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Shulgin, Oleksandr" <oleksandr(dot)shulgin(at)zalando(dot)de>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Patch to implement pg_current_logfile() function |
Date: | 2016-10-26 03:53:41 |
Message-ID: | 20161025225341.18e2a933@slate.meme.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, 25 Oct 2016 22:30:48 -0500
"Karl O. Pinc" <kop(at)meme(dot)com> wrote:
> Hope to provide more feedback soon.
Before I forget:
"make check" fails, due to oid issues with pg_current_logfile().
You're writing Unix eol characters into pg_log_file. (I think.)
Does this matter on MS Windows? (I'm not up on MS Windows,
and haven't put any thought into this at all. But didn't
want to forget about the potential issue.)
Now that pg_log_file contains multiple lines shouldn't
it be called pg_log_files?
In the docs, other functions that take optional arguments
show up as multiple rows. Attached is a new version of
my patch to the v6 patch which fixes this and supplies
a slightly better short description of pg_log_filename().
Regards,
Karl <kop(at)meme(dot)com>
Free Software: "You don't pay back, you pay forward."
-- Robert A. Heinlein
Attachment | Content-Type | Size |
---|---|---|
patch_pg_current_logfile-v6.diff.patchv2 | application/octet-stream | 4.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Karl O. Pinc | 2016-10-26 04:18:38 | Re: Patch to implement pg_current_logfile() function |
Previous Message | Venkata B Nagothi | 2016-10-26 03:32:44 | Re: patch proposal |