Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PAYARA-4078: Introduce ResouceInjectionServices #4209

Closed
wants to merge 1 commit into from

Conversation

@pdudits
Copy link
Contributor

pdudits commented Sep 9, 2019

Existing InjectionServices work well for classes within single module,
looking directly into deployment metadata, but interceptors need
standard JNDI based injection.

Naming exceptions during lookup are ignored, resulting in null
injection, which is compatible with previous behavior.

@pdudits

This comment has been minimized.

Copy link
Contributor Author

pdudits commented Sep 9, 2019

jenkins test please

@pdudits pdudits force-pushed the pdudits:payara-4078 branch 2 times, most recently from 9296a2f to 2a9a27e Sep 10, 2019
@pdudits

This comment has been minimized.

Copy link
Contributor Author

pdudits commented Sep 10, 2019

jenkins test please

@pdudits

This comment has been minimized.

Copy link
Contributor Author

pdudits commented Sep 10, 2019

Jenkins test please

@pdudits pdudits requested a review from jGauravGupta Sep 10, 2019
Existing InjectionServices work well for classes within single module,
looking directly into deployment metadata, but interceptors need
standard JNDI based injection.

Naming exceptions during lookup are ignored, resulting in null
injection, which is compatible with previous behavior.
@pdudits pdudits force-pushed the pdudits:payara-4078 branch from 2a9a27e to b0e6c5d Sep 10, 2019
@pdudits

This comment has been minimized.

Copy link
Contributor Author

pdudits commented Sep 10, 2019

Jenkins test please

@pdudits

This comment has been minimized.

Copy link
Contributor Author

pdudits commented Sep 11, 2019

Rolled up into #4210

@pdudits pdudits closed this Sep 11, 2019
@pdudits pdudits deleted the pdudits:payara-4078 branch Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.