From: | "Karl O(dot) Pinc" <kop(at)meme(dot)com> |
---|---|
To: | Gilles Darold <gilles(dot)darold(at)dalibo(dot)com> |
Cc: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, 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: | 2017-01-17 18:58:10 |
Message-ID: | 20170117125810.5cbc6940@slate.meme.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, 17 Jan 2017 19:06:22 +0100
Gilles Darold <gilles(dot)darold(at)dalibo(dot)com> wrote:
> Le 17/01/2017 à 03:22, Michael Paquier a écrit :
> > On Tue, Jan 17, 2017 at 2:21 AM, Karl O. Pinc <kop(at)meme(dot)com>
> > wrote:
> >> On January 15, 2017 11:47:51 PM CST, Michael Paquier
> >> <michael(dot)paquier(at)gmail(dot)com> wrote:
> >>> Also, I would rather see an ERROR returned to the user in case of
> >>> bad data in current_logfiles, I did not change that either as
> >>> that's the original intention of Gilles.
> I'm not against adding a warning or error message here even if it may
> never occurs, but we need a new error code as it seems to me that no
> actual error code can be used.
Seems to me the XX001 "data_corrupted" sub-category of
XX000 "internal_error" is appropriate.
https://www.postgresql.org/docs/devel/static/errcodes-appendix.html
As a dbadmin/sysadm I'd defiantly like to see something in the log if
you're going to raise anything user-side.
Regards,
Karl <kop(at)meme(dot)com>
Free Software: "You don't pay back, you pay forward."
-- Robert A. Heinlein
From | Date | Subject | |
---|---|---|---|
Next Message | Antonin Houska | 2017-01-17 19:05:25 | Re: PoC: Grouped base relation |
Previous Message | Andres Freund | 2017-01-17 18:53:46 | Re: Implement targetlist SRFs using ROWS FROM() (was Changed SRF in targetlist handling) |