On 15/12/2018 00:04, Alexey Bashtanov wrote: > I'd like to propose a patch to log bind parameter values not only when > logging duration, > but also on error (timeout in particular) or in whatever situation the > statement normally gets logged. > This mostly could be useful when the request originator doesn't log them > either, so it's hard > to reproduce the problem.
That's a reasonable problem to solve. > So I decided to cache textual representations on bind stage, > which is especially easy if the client uses text protocol. That sounds like a plausible approach. Have you done any performance measurements? In your patch, I would organize the changes to the portal API a bit differently. Don't change the signature of CreatePortal(). Get rid of PortalSetCurrentTop() and PortalClearCurrentTop(). Just have a global variable CurrentPortal and set it directly. And then change GetCurrentPortalBindParameters() to take a Portal as argument. This can all happen inside postgres.c without changing the Portal APIs. In fact, maybe don't use the Portal structure at all and just store the saved textualized values inside postgres.c in a static variable. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
