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.

Sign up for GitHub

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

Jump to bottom

Replace Lazy loading in ServiceFinder with Lazy ServiceLoader.Provider #4953

Open
jansupol opened this issue Jan 4, 2022 · 2 comments · May be fixed by #4999
Open

Replace Lazy loading in ServiceFinder with Lazy ServiceLoader.Provider #4953

jansupol opened this issue Jan 4, 2022 · 2 comments · May be fixed by #4999
Labels
JPMS

Comments

@jansupol
Copy link
Contributor

jansupol commented Jan 4, 2022

ServiceFinder is unable to parse module-info for providing the services. ServiceLoader since JDK 9 seems to be able to Lazy load classes and services using module-info.
@jansupol jansupol added the JPMS label Jan 4, 2022
@jansupol
Copy link
Contributor Author

jansupol commented Jan 4, 2022

Make sure the OSGi Services Loader still works in non-modularized OSGi environments.

@jbescos jbescos mentioned this issue Mar 4, 2022
Closed
@jbescos
Copy link
Member

jbescos commented Mar 7, 2022

Currently master branch is compiled with JDK 8 too, so this change will not compile.

Do I create a branch for Jersey 3 (which branch?), or we update Jenkinsfile to avoid JDK8?

@jbescos jbescos linked a pull request Mar 7, 2022 that will close this issue
Open
@jansupol jansupol linked a pull request May 18, 2022 that will close this issue
ServiceFinder defaults to ServiceLookupIteratorProvider #4999
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
JPMS
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ServiceFinder defaults to ServiceLookupIteratorProvider jbescos/jersey
2 participants
@jbescos @jansupol