On Tue, Nov 15, 2016 at 3:38 AM, Magnus Hagander <mag...@hagander.net> wrote: > Is there a reason why we don't log which relation triggered the snapshot too > old error when it happens? > > Since we do have Relation available in TestForOldSnapshot_impl(), shouldn't > we be able to include it? > > Having that in the log message would be very helpful, but is there a reason > why it wouldn't work?
I would probably not want to mess with the text of the error itself, in case any client-side software bases recovery on that rather than the SQLSTATE value; but including relation (and maybe the timestamp on the snapshot?) in detail or context portions seems like a great idea. -- Kevin Grittner EDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
