From: Tom Lane <tgl@sss.pgh.pa.us>
To: Alexander Lakhin <exclusion@gmail.com>
cc: Michael Paquier <michael@paquier.xyz>, a.kozhemyakin@postgrespro.ru,
        pgsql-bugs@lists.postgresql.org,
        Masahiko Sawada <sawada.mshk@gmail.com>, johncnaylorls@gmail.com
Subject: Re: BUG #18483: Segmentation fault in tests modules
In-reply-to: <439c9310-1d71-f2c6-b213-a9a2ea732ba9@gmail.com>
References: <18483-30bfff42de238000@postgresql.org>
 <Zl5GS6SHBXRAkl_c@paquier.xyz> <219049.1717792376@sss.pgh.pa.us>
 <ZmOJ3jfeltZQnN_M@paquier.xyz>
 <439c9310-1d71-f2c6-b213-a9a2ea732ba9@gmail.com>
Comments: In-reply-to Alexander Lakhin <exclusion@gmail.com>
	message dated "Sat, 08 Jun 2024 08:00:00 +0300"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <336832.1717859243.1@sss.pgh.pa.us>
Date: Sat, 08 Jun 2024 11:07:23 -0400
Message-ID: <336833.1717859243@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/336833.1717859243%40sss.pgh.pa.us>
Precedence: bulk

Alexander Lakhin <exclusion@gmail.com> writes:
> May I ask you about the project policy regarding such test modules
> failures?
> Do you think all of these should be fixed or it's ok to have some
> server-crashing modules in the tree?

I don't think there's a one-size-fits-all policy for them.  Since
we don't intend these modules to be installed, they don't really
need to work in any cases beyond the ones exercised in the tests.
Still, if it's easy to stop a misbehavior then we probably should.
The test_predtest problem was just a missing check-for-null so
I definitely regard that as worth fixing.  These other ones would
have to be investigated enough to figure out the size of the fix
before I'd venture an opinion.

			regards, tom lane