Hello
23.07.2018, 17:08, "Tom Lane" <tgl@sss.pgh.pa.us>:
> Sergei Kornilov <sk@zsrv.org> writes:
>> Please test with logging command tag %i in log_line_prefix. Extended protocol has three different messages, each
canbe canceled by timeout. But here is completely no parameters in PARSE and i did not change BIND in first patch.
>
> This patch scares me to death. It risks calling user-defined I/O
> functions in all sorts of weird places, particularly outside transactions,
> or in already-failed transactions, or with no ActiveSnapshot.
This is reason why i start thread with question how do it right way
As i wrote in beginning:
> i have no good idea how print ParamListInfo correctly. We can not use OidOutputFunctionCall in all cases, right?
Attached patch is just simple enough to illustrate one possible way.
I can further work with proper design, but i need idea how it should look.
> Separately from that concern: it appears to result in a substantial
> degradation of existing functionality in the places where you did
> s/errdetail/errdetail_log/. What was the reason for that?
This is my second question at thread beginning. Why used errdetail? We assume that the user wants to get their own
parametersback (if he set client_min_messages to LOG)?
regards, Sergei