Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Oct 21, 2021

Previously, the specification downright disallowed obtaining
a BeanManager in CDI Lite. This makes portability from CDI Full
to CDI Lite harder, which also manifests in the CDI TCK.

Wit this commit, CDI Lite applications may obtain a BeanManager,
but may only call methods inherited from BeanContainer.

Previously, the specification downright disallowed obtaining
a `BeanManager` in CDI Lite. This makes portability from CDI Full
to CDI Lite harder, which also manifests in the CDI TCK.

Wit this commit, CDI Lite applications may obtain a `BeanManager`,
but may only call methods inherited from `BeanContainer`.
@Ladicek
Copy link
Contributor Author

Ladicek commented Oct 21, 2021

I'm specifically adding @graemerocher as a reviewer for higher visibility. I think this is pretty straightforward to implement -- if you have a BeanContainer, implementing a BeanManager on top of it is simple: delegate BeanContainer methods to the BeanContainer, and throw an exception from other methods.

Copy link
Contributor

@manovotn manovotn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! This is going to save us some headaches in TCKs.

@Ladicek Ladicek merged commit 984b274 into jakartaee:master Oct 21, 2021
@Ladicek Ladicek deleted the beanmanager branch October 21, 2021 09:25
@Ladicek Ladicek added the Lite Related to CDI Lite label Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Lite Related to CDI Lite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants