> /* if we are doing RAISE, don't report its location */ > if (estate->err_text == raise_skip_msg) > return;
> is too simple, and this part should be fixed. This change can be done by on > plpgsql or libpq side. This is bug, and it should be fixed.
Doing this in libpq is utterly insane. It has not got sufficient context to do anything intelligent. The fact that it's not intelligent is exposed by the regression test changes that the proposed patch causes, most of which do not look like improvements.
I don't understand. There can be a overhead due useless transformation some data to client side. But all what it need - errcontext and errlevel is possible.
Another problem is that past requests to change this behavior have generally been to the effect that people wanted *more* context suppressed not less, ie they didn't want any CONTEXT lines at all on certain messages. So the proposed patch seems to me to be going in exactly the wrong direction.
The design I thought had been agreed on was to add some new option to plpgsql's RAISE command which would cause suppression of all CONTEXT lines not just the most closely nested one. You could argue about whether the behavior needs to be 100% backwards compatible or not --- if so, perhaps it could be a three-way option all, none, or one line, defaulting to the last for backwards compatibility.
I see a problem what should be default behave. When I raise NOTICE, then I don't need (don't would) to see CONTEXT lines, When I raise EXCEPTION, then I usually would to see CONTEXT lines.