Re: make -C src/test/isolation failure in index-killtuples due to btree_gist
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Michael Paquier <michael@paquier.xyz>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-10-30T15:24:55Z
Lists: pgsql-hackers
Hi, On 2025-10-28 16:55:49 +0900, Michael Paquier wrote: > On Fri, Aug 22, 2025 at 07:46:43PM -0400, Andres Freund wrote: > > That would be the easiest fix - but I'm starting to wonder if it shouldn't > > just be its own test module, as annoying as the boilerplate for that is. > > > > While the test improved code coverage for the various indexes noticeably, I > > did subsequently realize that the new test doesn't end up testing the recovery > > path :(. Better than nothing, but having any coverage of those paths might be > > worth the boilerplate and the runtime overhead of a test module :/ > > Adding just an EXTRA_INSTALL to isolation's Makefile would not work, > no? How precisely to get the contrib module installed seems like a boring detail. It's not hard to do that, if we decide that that's the way to go, even without EXTRA_INSTALL. > How about a new test/modules/gist/ then? Well, it tests multiple index types, not just gist. I guess test/modules/indexes/ or so should work? Not sure about plural or not. Greetings, Andres Freund
Commits
-
Move isolation test index-killtuples to src/test/modules/index/
- e429c3cecb4a 19 (unreleased) landed