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
chore: create Jandex index #1367
Conversation
This makes lookup of classes faster in environments that use Jandex, providing better performance in some environments
@rohanKanojia tests passed but I don't know why the |
ok to test |
@gastaldi : we have to whitelist PRs first ;-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, Thanks 👍
I'm just not clear about the use case for adding this plugin. I looked into jandex-plugin-website here: http://docs.wildfly.org/jandex-maven-plugin/1.0.1/source-repository.html . How much build time is used additionally to build up the index? Can we put this plugin in release profile rather than main build?
Also, would be awesome if you could add a line about this change to CHANGELOG :-)
Ok to test |
[merge] |
@gastaldi: I like the way you think! |
This makes lookup of classes faster in environments that use Jandex, providing better performance in some environments