Use CDI discovery mode annotated

Question

Use CDI discovery mode annotated

johnament opened this issue 8 years ago · 5 comments

Annotated discovery is better than all discovery. The usage of all assumes your runtime has a hierarchical class loader, and not a flat class loader. In order for it to be usable in both setups, I would recommend switching beans.xml to annotated and adding proper annotations to all classes.

Answer 1 · 2016-08-17T15:53:00.000Z

Looks like this is done, so can be closed

Answer 2 · 2016-08-18T01:34:27.000Z

There's still a few places where annotations arne't used. its actually demonstrating non-portable behavior IMHO. Would be good to clarifications on these.

https://github.com/hammock-project/microprofile-samples

Answer 3 · 2016-08-19T02:08:37.000Z

Empty beans.xml (meaning CDI 1.0 discovery behavior): https://github.com/microprofile/microprofile-samples/blob/master/microprofile-sample-canonical/src/test/java/io/microprofile/sample/canonical/rest/TopCDsEndpointTest.java#L74

Answer 4 · 2016-08-29T21:43:00.000Z

Using annotated the application fails to work on Payara/GlassFish as TopCDsEndpoint does not have a bean defining annotation. I just opened a pull request to change it back ;-) #15

Answer 5 · 2016-09-23T15:16:37.000Z

This should be fixed now. @smillidge can we add a payara profile for this test?