| From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
|---|---|
| To: | Alexander Pyhalov <a(dot)pyhalov(at)postgrespro(dot)ru> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alexander Korotkov <aekorotkov(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Ronan Dunklau <ronan(dot)dunklau(at)aiven(dot)io> |
| Subject: | Re: SQLFunctionCache and generic plans |
| Date: | 2025-03-13 16:59:14 |
| Message-ID: | CAFj8pRD0diy5VDP=8B9au5sk_yRknCWqZx35VniBtP_OA4aJNw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi
I am checking last patches
Maybe interesting change is the change of error message context
QUERY: SELECT public.dep_req2() || ' req3b'.
-CONTEXT: SQL function "dep_req3b" during startup
+CONTEXT: SQL function "dep_req3b" statement 1
almost all SQL functions have just one statement, so showing the number of
the statement looks useless
(until now, I didn't see multiple statements SQL function) ,
we lost the time info "during startup". Maybe the error message can be
enhanced more like plpgsql,
instead of statement numbers, the lines or positions should be displayed.
The changing context text can be done in a separate patch - and in this
moment, we
can use old behaviour.
Regards
Pavel
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jacob Champion | 2025-03-13 17:29:49 | Re: [PATCH] pg_stat_activity: make slow/hanging authentication more visible |
| Previous Message | Andres Freund | 2025-03-13 16:56:09 | Re: [PATCH] pg_stat_activity: make slow/hanging authentication more visible |