search

quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
13.44k stars 2.58k forks source link

Chicken and egg: virtual-threads/jms-virtual-threads depends on io.quarkiverse.artemis #38639

Open gsmet opened 6 months ago

gsmet commented 6 months ago

Having the core repository depending on a Quarkiverse extension is problematic IMO. That's a chicken and egg issue and if at some point we have changes of the magnitude of the Jakarta EE 10 changes, it's going to be in the way.

I think this test should either be adjusted to not use this module or moved to io.quarkiverse.artemis.

Let me know if there are strong reasons not to do it.

/cc @cescoffier

quarkus-bot[bot] commented 6 months ago

/cc @cescoffier (virtual-threads), @ozangunalp (virtual-threads)

cescoffier commented 6 months ago

We would need to move it as the JMS connector requires an implementation of the ConnectionFactory.

ozangunalp commented 6 months ago

In any case, we don't have a reactive-messaging-jms extension. I think we can remove the virtual-threads/jms-virtual-threads from the quarkus repo. (Not sure where to put)