On Fri, Mar 5, 2021 at 1:37 PM Michael Paquier <mich...@paquier.xyz> wrote: > > On Fri, Mar 05, 2021 at 01:21:01PM +0100, Daniel Gustafsson wrote: > > On 5 Mar 2021, at 08:04, Michael Paquier <mich...@paquier.xyz> wrote: > >> FWIW, I would vote to nuke it from all those places, reducing a bit > >> pg_stat_get_activity() while on it. Keeping it around in the system > >> catalogs may cause confusion IMHO, by making people think that it is > >> still possible to get into configurations where sslcompression could > >> be really enabled. The rest of the patch looks fine to me. > > > > Attached is a version which removes that as well. > > Peter, Magnus, any comments about this point?
We've broken stats views before. While it'd be nice if we could group multiple breakages at the same time, I don't think it's that important. Better to get rid of it once and for all from as many places as possible. -- Magnus Hagander Me: https://www.hagander.net/ Work: https://www.redpill-linpro.com/
