On Tue, Oct 30, 2018 at 2:39 AM Iwata, Aya <iwata....@jp.fujitsu.com> wrote: > I create a first libpq trace log patch.
Couple additional thoughts from a read-through of the patch: - PQtrace() and the new trace-logging machinery overlap in some places but not others -- and if both are set, PQtrace() will take precedence. It seems like the two should not be separate. - It isn't immediately clear to me how the new information in the logs is intended to be used in concert with the old information. Maybe this goes back to the comments by Tom and Robert higher in the thread -- that an overhaul of the PQtrace system is due. This patch as presented would make things a little worse before they got better, I think. That said, I think the overall idea -- application performance information that can be enabled via the environment, without requiring debugging privileges on a machine or the need to manually correlate traces made by other applications -- is a good one, and something I would use. --Jacob
