You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by "zhfeng (via GitHub)" <gi...@apache.org> on 2023/03/22 00:17:32 UTC
[GitHub] [shardingsphere] zhfeng opened a new issue, #24726: Add jandex-maven-plugin to generate idx for classes
zhfeng opened a new issue, #24726:
URL: https://github.com/apache/shardingsphere/issues/24726
## Feature Request
**For English only**, other languages will not be accepted.
Please pay attention on issues you submitted, because we maybe need more details.
If no response anymore and we cannot make decision by current information, we will **close it**.
Please answer these questions before submitting your issue. Thanks!
### Is your feature request related to a problem?
I'd like to support [quarkus-shardingsphere-jdbc](https://github.com/quarkiverse/quarkus-shardingsphere-jdbc) in native building with GraalVM. It needs to find some classes and register them for reflection and service provider at build time. Because shardingsphere is heavily using the SPI mechism, it would be very handy to leverage on [jandex](https://github.com/smallrye/jandex) to find all impementor of a SPI interface.
### Describe the feature you would like.
Add [jandex-maven-plugin](https://github.com/wildfly/jandex-maven-plugin) to generate idx during `process-classes`. There would be a `META-INF/jandex.idx` in jar, so it could increase the size of artifact package by about **5%**. I know this is a trade-off and it should be very helpful for my working on native support with quarkus-shardingsphere-jdbc. So I'm very appercaited if this could be accepted by community.
Thanks in advance!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] zhfeng commented on issue #24726: Add jandex-maven-plugin to generate idx for classes
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on issue #24726:
URL: https://github.com/apache/shardingsphere/issues/24726#issuecomment-1478833590
I'm taking a look at https://github.com/smallrye/jandex which is Apache-2 License. So it should be OK.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] zhfeng commented on issue #24726: Add jandex-maven-plugin to generate idx for classes
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on issue #24726:
URL: https://github.com/apache/shardingsphere/issues/24726#issuecomment-1478796075
Hmm, the only issue is that [jandex-maven-plugin](https://github.com/wildfly/jandex-maven-plugin) is licensed with GPL-3.0 and I'm not very sure that we can use it only in building the project?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] zhfeng commented on issue #24726: Add jandex-maven-plugin to generate idx for classes
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng commented on issue #24726:
URL: https://github.com/apache/shardingsphere/issues/24726#issuecomment-1478828565
Sorry, it looks like a license issue that we can not use [jandex-maven-plugin](https://github.com/wildfly/jandex-maven-plugin) and package with idx files. see https://www.apache.org/legal/resolved.html#prohibited
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] zhfeng closed issue #24726: Add jandex-maven-plugin to generate idx for classes
Posted by "zhfeng (via GitHub)" <gi...@apache.org>.
zhfeng closed issue #24726: Add jandex-maven-plugin to generate idx for classes
URL: https://github.com/apache/shardingsphere/issues/24726
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] terrymanu closed issue #24726: Add jandex-maven-plugin to generate idx for classes
Posted by "terrymanu (via GitHub)" <gi...@apache.org>.
terrymanu closed issue #24726: Add jandex-maven-plugin to generate idx for classes
URL: https://github.com/apache/shardingsphere/issues/24726
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org