kfaraz commented on PR #18147: URL: https://github.com/apache/druid/pull/18147#issuecomment-2983683149
> I would really like the separate module - but placing these tests where they should belong will be more in front of someone working on that part...so I think its better to put it inside an existing module - it might be possible to create a separate source set for it; but that would only be a logical isolation (probably with no real benefits) Thanks for the feedback, @kgyrtkirk ! I feel the same. > we could have a module which loads multiple (or all) extensions so that more complicated scenarios could be tested....actually that was the reason I created quidem-ut - maybe we could rename that module so that the multi-extension simulation tests could live there along with the qtest stuff? Yeah, that would be nice! I originally thought we could put it into `integration-tests` module but the quidem UT module is more closely related. We could just call that module `simulation-tests` since quidem UTs are also essentially a type of simulation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
