Robert Haas <robertmh...@gmail.com> writes: > On Sun, Dec 10, 2017 at 11:55 AM, Tom Lane <t...@sss.pgh.pa.us> wrote: >> There seem to be two ways we could look at this. One is that the >> new test case just needs to be rejiggered to avoid unstable output >> ("\set VERBOSITY terse" would be the easiest way). But there is >> also room to argue that it's bad that plpgsql produces error reports >> that vary depending on the phase of the moon, which is pretty much >> what this would look like in the field --- cache flushes will occur >> unpredictably in most application environments.
> I am inclined toward the latter view. Yeah, me too. I'll see about patching exec_eval_simple_expr() to provide a context line that matches SPI's. Seems like a HEAD-only change though, as this will result in visible behavior change in the typical case. regards, tom lane