Be more consistent about the CDI qualifiers we apply to Hibernate Search/ORM beans

[marko-bekhta]

Hibernate Search processor always adds the named qualifier:

https://github.com/yrodiere/quarkus/blob/888266f11c87a961fc9ae28c725f71b94fc2b475/extensions/hibernate-search-orm-elasticsearch/deployment/src/main/java/io/quarkus/hibernate/search/orm/elasticsearch/deployment/HibernateSearchElasticsearchCdiProcessor.java#L68-L69

while Hibernate ORM:

https://github.com/yrodiere/quarkus/blob/888266f11c87a961fc9ae28c725f71b94fc2b475/extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateOrmCdiProcessor.java#L350-L353

adds them conditionally. Let's be consistent and always add them.

• See also Use Arc features in Hibernate extensions for eager startup and active/inactive #48783 (comment)

[quarkus-bot]

/cc @Ladicek (arc), @manovotn (arc), @mkouba (arc)

[yrodiere]

Agreed. This should be easy to fix, PRs welcome.

EDIT: Ideally the PR would come with a test though :) see *CDI*Test in extensions/hibernate-orm/deployment/src/test/java

[Aryant-Tripathi]

@yrodiere Can I work on this issue, I am new to quarkus and want to explore by learning.

[marko-bekhta]

Hey @Aryant-Tripathi sure 🙂
You can work on this one and open a pull request once you have the changes ready.
If you are new to Quarkus, I'd encourage you to have a look at the contributing guide to help you set up the env and learn how to make your first steps around Quarkus: https://github.com/quarkusio/quarkus/blob/main/CONTRIBUTING.md

[Aryant-Tripathi]

Hey @yrodiere @marko-bekhta I know this is a very small change, I have made changes but I am not sure about creating destination of the test case which file should contain. Can you explain help me a little about how can i test this and where i can create the test case.

I am really sorry asking about asking this dumb question 🙂

[marko-bekhta]

Hey @Aryant-Tripathi 👋🏻

you can take a look at

quarkus/extensions/hibernate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/multiplepersistenceunits/MultiplePersistenceUnitsCdiEntityManagerTest.java

Line 25 in b03a2f8

public class MultiplePersistenceUnitsCdiEntityManagerTest {