| 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-11-04 20:07:52 |
| Message-ID: | 20161104150752.28ef1cc1@slate.meme.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, 4 Nov 2016 16:58:45 +0100
Gilles Darold <gilles(dot)darold(at)dalibo(dot)com> wrote:
> I attached a v12 patch
Attached is a comment patch which improves the comment
describing CURRENT_LOG_FILENAME. It's been bugging me.
I should have made this change long ago when I looked
at all the other code comments but neglected to.
The comment now matches other documentation.
This applies on top of your v12 patch.
I'm thinking through the v12 patch and email.
I'm in general agreement but want to make sure
I really understand all the code paths.
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-v12.diff.symbolcomment | application/octet-stream | 788 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Emre Hasegeli | 2016-11-04 20:14:35 | Re: btree_gin and btree_gist for enums |
| Previous Message | Thomas Munro | 2016-11-04 19:42:47 | Re: Gather Merge |