On Thu, 2021-09-16 at 02:22 -0700, Andres Freund wrote: > I pushed this. The only substantive change I made is that I moved the > MyBackendType == B_BACKEND check into a new pgstat_should_report_connstat(), > and called that from pgstat_report_connect() and > pgstat_report_disconnect(). Otherwise we'd report connect/disconnect for > walsenders, but not the session time, which seems even weirder.
Oh, right, I didn't think of that. Thanks for all the work you put into this. > I'm not convinced that ignorign walsenders is a good policy decision, but I > think that's a fair judgement call, and we can evolve that in future > releases... The use case that is on my mind is to figure out the percentage of time when transactions are idle to be able to estimate the connection pool size. walsenders, for which you typically don't want to use a connection pool, would skew the statistics. Yours, Laurenz Albe
