On Mon, Mar 04, 2024 at 05:17:52AM +0100, Jelte Fennema-Nio wrote: > I noticed this was committed, and took a quick look. It sounds very > useful for testing Citus to be able to use injection points too, but I > noticed this code is included in src/test/modules. It sounds like that > location will make it somewhat hard to install. If that's indeed the > case, would it be possible to move it to contrib instead?
One problem with installing that in contrib/ is that it would require more maintenance as a stable and "released" module. The aim of this module is to be more flexible than that, so as it is possible to extend it at will even in the back branches to be able to implement features that could help with tests that we'd want to implement in stable branches. I have mentioned that on a separate thread, but adding more extension maintenance burden while implementing complex tests does not sound like a good idea for me in the long-run. Perhaps we could consider that as an exception in "contrib", or have a separate path for test modules we're OK to install (the calls had better be superuser-only if we do that). Another thing with the backend support of injection points is that you could implement your own extension within citus, able to do what you mimic this in-core module, and get inspiration from it. Duplication is never cool, I agree, though. -- Michael
signature.asc
Description: PGP signature