Date: Sat, 21 Aug 2021 09:37:26 +0900
From: Michael Paquier <michael@paquier.xyz>
To: Ronan Dunklau <ronan.dunklau@aiven.io>
Cc: Magnus Hagander <magnus@hagander.net>,
	pgsql-hackers@lists.postgresql.org,
	Alvaro Herrera <alvherre@alvh.no-ip.org>
Subject: Re: Proposal: More structured logging
Message-ID: <YSBKT8kUMgssml3K@paquier.xyz>
References: <202108191450.7vik45yrlwle@alvherre.pgsql>
 <6062364.BDNAq9uPBk@aivenronan>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="vazQhqT0DeHGnteT"
Content-Disposition: inline
In-Reply-To: <6062364.BDNAq9uPBk@aivenronan>
Archived-At: 
 <https://www.postgresql.org/message-id/YSBKT8kUMgssml3K%40paquier.xyz>
Precedence: bulk


--vazQhqT0DeHGnteT
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Aug 20, 2021 at 11:35:29AM +0200, Ronan Dunklau wrote:
> Michael, your jsonlog module already fullfills this need. Is it something=
 that=20
> should be merged into our tree ?

Yes, there is nothing technically preventing to have this stuff in
core, of course, and that would even take care of the issues in
detecting if the piping protocol should be used or not.

Now, the last time this was proposed, there was a lot of drawback
because the presence of the JSON keys increases significantly the size
of the logs compared to CSV, and usually users care a lot about the
size of the log files.  I would support the presence of JSON format
for the logs in core, FWIW.

The module is here, for reference:
https://github.com/michaelpq/pg_plugins/tree/master/jsonlog
--
Michael

--vazQhqT0DeHGnteT
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmEgSsYACgkQnvQgOdby
QH35/A//d8Gxg0CZ9r5cP8E+N5IX7luqFKaUg6BFYN6aH75P05NYbo3q3nfMKiYl
Ug1+0rAMxGx07KJkZOL+pA38vgjxm9Zz2p05S0MGWPOrgfIQorAb98j1Kp/vE5mh
BzLS8kndA7+U006KOfcXGziXGFitXTsL82lCV6+866Pou+o1FqLGS76RMtY+0yXm
ozC+33wHuWO2rTfXvfc5bIVpC56lIY0G6UXrFoT7h8CYUOWY3oWufMzT5IC1GKZN
Y+ucUgXwfZX4kzAOmFB5+cn+1Pq0vVYz/lLQRyTaLkeSG41CmmLLXtzi64pUee8a
ge30LLWuviOCL6C39Gu8b4kZjSs5kGBQ4Adtvd0yLvC00KSJlVe0WutEhhsrh3H5
tNeyKZozL+lyy+KkFIVcB5yvP8thMglyv2IKlxGlth2lEzBfZnjC0X4tpL/1VL9X
6GgAbLfWEhzaIFqxJ5SQnadkzMOwUB1/5VH1a7Q5kwOPfrOoBfZbTiKBm+XaCe9G
O36db3Tz3rhNUfRId6QKKRncOvuNFnH0Vw66Ucvi2n+gSCjasXifMTfClYvu4HGb
vz/tCY/bbFNH7GwQfWCSqXmM6hokIQK9CxYCTCYOtEBC3p5ks1tIlgR0teTsswFZ
IQsB3krCt+pGIYZDVgctD+7m4pfAzMLDjmbFhSEoDHdrhJimnZk=
=hIBw
-----END PGP SIGNATURE-----

--vazQhqT0DeHGnteT--