Feedback-ID: i0fe9450f:Fastmail
Date: Tue, 16 May 2023 16:48:35 +0900
From: Michael Paquier <michael@paquier.xyz>
To: "Drouvot, Bertrand" <bertranddrouvot.pg@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: Autogenerate some wait events code and documentation
Message-ID: <ZGM1U5D5EuBUbasi@paquier.xyz>
References: <ZEdh2+46GuhCqW6b@paquier.xyz>
 <e8a1e3c2-26e5-8a7a-f821-eee92ab0c8e2@gmail.com>
 <64fc361e-f4ed-45eb-0cfb-c8110872530d@gmail.com>
 <ZEoSlv9HxURD8Uxw@paquier.xyz>
 <e59598e3-3630-e90d-8985-1833f8d5aede@gmail.com>
 <ZE8A8IIJ5GXfIKv8@paquier.xyz>
 <124bdcea-bdc8-e3d2-2357-0c6a857bc557@gmail.com>
 <ZFW6FTMIH00502RU@paquier.xyz>
 <ZFW9UYQFyftKpoaH@paquier.xyz>
 <106e8d0a-0820-687a-7da5-3143e1f5b05b@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="YG9W6nBIFi3x4MLB"
Content-Disposition: inline
In-Reply-To: <106e8d0a-0820-687a-7da5-3143e1f5b05b@gmail.com>
Archived-At: <https://www.postgresql.org/message-id/ZGM1U5D5EuBUbasi%40paquier.xyz>
Precedence: bulk


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

On Mon, May 15, 2023 at 06:45:23PM +0200, Drouvot, Bertrand wrote:
> On 5/6/23 4:37 AM, Michael Paquier wrote:
>> On Sat, May 06, 2023 at 11:23:17AM +0900, Michael Paquier wrote:
>>>> I'll look at v7 when the v17 branch opens and propose the separate pat=
ch
>>>> mentioned above at that time too.
>>>=20
>>> Thanks, again.
>>=20
>> By the way, while browsing through the patch, I have noticed two
>> things:
>> - The ordering of the items for Lock and LWLock is incorrect.
>=20
> Oh right, fixed in V8 attached (moved the sort on the third column
> instead of the second which has always the same content "WAIT_EVENT_DOCON=
LY"
> for Lock and LWLock).

Ah, I didn't notice that.  Makes sense.

>> - We are missing some of the LWLock entries, like CommitTsBuffer,
>> XactBuffer or WALInsert, as of all the entries in
>> BuiltinTrancheNames.
>=20
> Yeah, my bad. Fixed in V8 attached.

BufFileTruncate and BufFileWrite have an incorrect order in HEAD's
monitoring.sgml (will fix in a minute for 16~).  So your patch fixes
that.

PgStatsDSA and PgStatsData are reversed in your patch compared to
HEAD, actually, based on the way perl sees fit to do its ordering by
giving priority to upper-case characters.  Same for RelCacheInit and
RelationMapping, or even SInvalRead/SInvalWrite being now before the
"Serial" family.  Worse, the tables LWLock and Lock are in an
incorrect order as well with the patch.  We'd better be a bit more
verbose with the sort step, I think..  perl does not handle well
sorting with collations from what I recall, but we could use uc() with
a block sort to force the operation to be case-insensitive, like "sort
{uc($a) cmp uc($b)}".  That needs to be applied here, I guess:=20
+# Sort the lines based on the third column
+my @lines_sorted =3D
+  sort { (split(/\t/, $a))[2] cmp(split(/\t/, $b))[2] } @lines;

And it looks like you'd need to apply uc() on each [2] element.  I
would add a comment about this detail, as well.

No entries are missing, after comparing what's generated by the patch
with the contents of HEAD.

Small nit-ish question: waiteventnames.sgml or wait_event_types.sgml?
Same for generate-waiteventtypes.pl?

>> My apologies for not noticing that earlier.  This exists in v6 as much
>> as v7.
>=20
> No problem at all and thanks for the call out!

FWIW, I would have posted two patches, one with the refactoring of
done in [1], and a second that switches to the automation, to make
clear the preparatory step.

[1]: https://www.postgresql.org/message-id/c6f35117-4b20-4c78-1df5-d3056010=
dcf5@gmail.com
--
Michael

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

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

iQIzBAABCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmRjNVMACgkQnvQgOdby
QH22ig/+MP9wqll6mqc+yyBrEw8GQIvFzQH5JM4z+Jo0yVNFuHONmHnYxz3ryHBN
Kz93gTbbSkhOO/Acl85xkdmumwHB47KM9mqARYz1TcKQwluopsxaWTjwqrMNiWv3
9S9JmlOa0MyOv03uySf/tKDMHC4RAo/eJ7x3pZHSPNqF2P9jzAOb6LJa1LqMBtMk
UMo10Frv+hQQsE8wfunydOVusTuqkvF2PhhaWXHi8D4EJf7oOLG1i6jOjtfH/s/L
onVqLSG1GLqJXXUY5pOijtCBIWyE8VY8TQ7E4oGaNIMzKryPuQkteP9rw3+s944o
lK/mw2VjKDWW1MZs0k9ZUZK7D+BJjwuHRbCBG89wR4h8YZ6gKLOFqWzi/N14yXpZ
nJAaF5EMK+cahPcBgxhnwaeewPNAX+5dEby9K4/nCM/QVlVQVmAT5rxb6jydj9Op
M2jYmNYtgziwiRaPXnmRJDJJVzJPI2LzqWgMVkPMRaBDUL65URTAx3cUeiipna/o
Dz5QgDFCShSN+ioMvA7VfSs/hd6am4tZnCfz0rU7BcueNd618sOYCtAFA3DWHvnK
oOeHO/KVDjwipH0YcOPC4H5TZtiuvFLS17JpyTyeHtv+fSymcy9kQR68Tu+Teav6
rLfPBfyxXBpk78ui1AP2PR+u8KLBGumh2/NUlUBJ8yJm4qJFz7I=
=a+BG
-----END PGP SIGNATURE-----

--YG9W6nBIFi3x4MLB--