Feedback-ID: i0fe9450f:Fastmail
Date: Thu, 17 Sep 2026 17:03:38 +0900
From: Michael Paquier <michael@paquier.xyz>
To: Alexandre Felipe <o.alexandre.felipe@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: pg_regress: schedule multi-line test groups
Message-ID: <aque2pNHYkJC5iyR@paquier.xyz>
References: 
 <CAE8JnxObF4iYtNA-3w+JDNT2YsbS5YMyGiPMnYU0Gd0CtUOMGQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="mXaIrcoWWrvI5GVB"
Content-Disposition: inline
In-Reply-To: 
 <CAE8JnxObF4iYtNA-3w+JDNT2YsbS5YMyGiPMnYU0Gd0CtUOMGQ@mail.gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/aque2pNHYkJC5iyR%40paquier.xyz>
Precedence: bulk

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

On Thu, Sep 17, 2026 at 08:57:51AM +0100, Alexandre Felipe wrote:
> This was motivated by two rebases on the SLOPE patch [1] this week due to
> changes in parallel_schedule. Also I find much easier to parse the list
> using this syntax, if we stick with one test per line we get the benefit =
of
> counting tests by looking at the line numbers.
>=20
> I used a `goto` to keep the number of touched lines small, and facilitate
> your review.

I'd +1 something like that in the schedule files, even if we only
limit the number of tests per line to be up to 20.

+test:
+    boolean
+	char
+	name

At least, let's make the whitespace vs tab policy consistent.  ;)
--
Michael

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

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

iQIzBAEBCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAmqrntoACgkQnvQgOdby
QH1Z9A/+OCw9zYo2Upicsd0okjJKdBu9jR4EHGL+XqgZiik/+OG82goHLUn9vEsE
lPfpWouwqhYslKaV4jhIzOzmsciS16fNKFmy1enLnEY3XO5nlC36ixUmXSvJibDo
Pd1GZ1MDHVLxji4vkk6qA948JSFHaRyRgPR1Ys6fc2ByyvPHEK+iz85W8RC4pbVz
0H+Zh/CR+Uy1q4kTC+xn+94K9jTVSiFrk4nx6HzFEapcWD7RqmjjWaWt6s8lxdWK
9aThh1/1umGC1axbVZ3fn4tEpCcAIxfXRVZ/Ns+MbrnnmrLTDWyLzRvGdrnhqERV
/pVHWACvstuaal4HOhB4KbkY++uhtovqbhE0f11BhuMvt7pOWp8AbpiRckyrG3F2
w6dAzq6wFUhBbP5nMqmlt8Eu5R722amlg9m+Iv93bhfoO6zMdMz9DMIHkZ72scb/
YfFBnO3q/zOWpb7Pmpksu9XiDWGv7LoFwfh2lfPM1TqVK+SDc9zDORQiwn1s5Mkd
XxdxKUJukiLoVLA6zP9aR7eUZHf1D3YY2dBZlpFWyzLl6yra2kD8aYeSp9maHr9n
6jjHVyD0fX+FJ3EIaIlWtTge+xu+Uf7iJZZTec443y73o0BpxYGMqNeJqYoQ7yEW
Fuzq9qSTGD4UEdtKWpFGy1fQ1fTTK2OClCPP99p9+bPWjvmKlns=
=G1we
-----END PGP SIGNATURE-----

--mXaIrcoWWrvI5GVB--