vert-x3 / vertx-ignite

Apache License 2.0
35 stars 28 forks source link

Have you considered adding support for GridGain? #130

Closed zhouzhou19950825 closed 1 year ago

zhouzhou19950825 commented 1 year ago

Read me

At present, ignite's support for jdk17 is not very good, but gridgain's support is good. Have you considered adding support for GridGain?

zyclonite commented 1 year ago

vertx-ignite runs on jdk-19 in production without issues...

do you mean commercial support? or do you have specific issues with jdk-17 where you need support?

zhouzhou19950825 commented 1 year ago

@zyclonite On jdk17, need to add the corresponding vmoptions to start

--add-opens=java.base/jdk.internal.misc=ALL-UNNAMED --add-opens=java.base/sun.nio.ch=ALL-UNNAMED --add-opens=java.management/com.sun.jmx.mbeanserver=ALL-UNNAMED --add-opens=jdk.internal.jvmstat/sun.jvmstat.monitor=ALL-UNNAMED --add-opens=java.base/sun.reflect.generics.reflectiveObjects=ALL-UNNAMED --add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED --add-opens=java.base/java.io=ALL-UNNAMED --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.locks=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.math=ALL-UNNAMED --add-opens=java.sql/java.sql=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED

zyclonite commented 1 year ago

correct, the minimal args i am using are

--add-opens=java.base/java.nio=ALL-UNNAMED 
--add-opens=java.base/java.util=ALL-UNNAMED 
--add-opens=java.base/java.lang.invoke=ALL-UNNAMED 
--add-opens=java.base/java.lang=ALL-UNNAMED
zhouzhou19950825 commented 1 year ago

@zyclonite For the completeness of the document, the vm parameter content is added https://github.com/vert-x3/vertx-ignite/pull/131

zyclonite commented 1 year ago

thank you for your contribution

tsegismont commented 1 year ago

Thank you @zhouzhou19950825 for #131

@zyclonite I believe we can close this one now, can't we?