Indeed, running the server with JDK11 fixed the problem. The client Groovy script could be still run using JDK17.
Thanks for the tip. Cheers, Paul. On Mon, Dec 9, 2024 at 2:25 PM Imba Jin <j...@apache.org> wrote: > > Hi, Paul > > Glad to see your feedback, a quick reply for it: > > 1. Currently, the `gremlin-server` version 3.5.x in HG doesn't support run in > Java 17(Seems supported in Gremlin 3.7.x?), although it can be compiled well, > but it will encounter errors during runtime(could try to run it with Java11). > > 2. Our latest image is directly bound to the code of the master branch, so it > depends on the pom version of the master code. I will soon improve the > default description in the document/README to let everyone use a fixed > release tag instead of the latest image, to avoid users ignoring the "NOTE" > information later > > Thank you again for your patient feedback, we will continue to improve it~ > > Imba Jin > > On 2024/12/08 21:39:22 Paul King wrote: > > Re: point 2 (I may be missing something but to me it looks like new > > jars in the docker image): > > > > $ docker ps > > CONTAINER ID IMAGE COMMAND CREATED > > STATUS PORTS > > NAMES > > 4ce4ddbaf9d1 hugegraph/hugegraph "/usr/bin/dumb-init …" 2 > > minutes ago Up 2 minutes 0.0.0.0:8080->8080/tcp, :::8080->8080/tcp > > graph > > > > $ docker exec -it graph bash > > > > root@4ce4ddbaf9d1:/hugegraph-server# ls lib > > ... > > hugegraph-api-1.5.0.jar > > hugegraph-cassandra-1.5.0.jar > > hugegraph-common-1.5.0.jar > > hugegraph-core-1.5.0.jar > > hugegraph-dist-1.5.0.jar > > hugegraph-hbase-1.5.0.jar > > hugegraph-hstore-1.5.0.jar > > hugegraph-mysql-1.5.0.jar > > hugegraph-palo-1.5.0.jar > > hugegraph-postgresql-1.5.0.jar > > hugegraph-rocksdb-1.5.0.jar > > hugegraph-rpc-1.5.0.jar > > hugegraph-scylladb-1.5.0.jar > > ... > > > > Cheers, Paul. > > > > On Sun, Dec 8, 2024 at 11:07 PM Yuchen Ding <vgalax...@apache.org> wrote: > > > > > > Hi Paul, > > > > > > Thank you for your feedback and support! > > > > > > Regarding the caveats: > > > > > > 1. For the issue with the Gremlin examples not running as expected, this > > > may be related to recent changes we made. We appreciate your observation > > > and encourage you to report this in our main repo issue so we can > > > reproduce and address it promptly. > > > 2. The release version shown in the README example is because we made a > > > bulk replacement of the version number while modifying the pom file. > > > However, we did not actually push it (image) before the release done. > > > Thank you for pointing this out, we’ll ensure the README content is only > > > updated after the vote finished to avoid confusion. > > > 3. The swagger-ui files are indeed under the Apache-2.0 license, but we > > > overlooked standalone files like swagger-ui.css while focusing on jar > > > dependencies. We’ll address it & check similar files. > > > > > > Thanks again for your detailed review and valuable insights! > > > > > > Best regards, > > > VGalaxies > > > > > > On 2024/12/08 03:02:38 Paul King wrote: > > > > +1 (binding) with caveats > > > > > > > > I checked: > > > > * Checksum and GPG signatures are OK > > > > * LICENSE, NOTICE, DISCLAIMER exist > > > > * Incubating in name > > > > * No unexpected binary files in src tar.gz files (I checked mostly the > > > > main tar.gz file) > > > > * Server from convenience binaries ran fine on macOS 15.1.1 > > > > * Compiled from source was successful with Java 17.0.3, Maven 3.9.9 > > > > * I ran the code from my groovy-graphdb blog which uses Groovy and > > > > HugeGraph: > > > > https://groovy.apache.org/blog/groovy-graph-databases > > > > https://github.com/paulk-asert/groovy-graphdb/tree/main/hugegraph > > > > > > > > Caveats: > > > > * When running the code from the blog post mentioned above, the server > > > > didn't seem to have Gremlin support even though I thought my > > > > gremlin-server.yaml file and rest-server.properties files did have it > > > > configured (might be something I missed locally): > > > > gremlin-groovy is not an available GremlinScriptEngine > > > > Non-gremlin examples ran fine. > > > > * When running against the current docker image the examples ran fine > > > > (though I was surprised that the docker image was 1.5.0 even though > > > > the vote hasn't passed as yet). I believe the docker image should only > > > > be published after the vote has passed. I know docker images can be > > > > updated at any time, so I didn't see this as blocking. > > > > * While the work done for license/notice files looks excellent in > > > > general, there still seems to be some things missing, e.g. > > > > swagger-ui.css has MIT license, but that doesn't seem to be reflected > > > > in current license files. Diligent observers of the convenience binary > > > > release would find the MIT License info, so I didn't see this as > > > > blocking, but this should ideally be fixed for the next release. There > > > > might be more files like this where things need updating, that file is > > > > just the first I noticed. > > > > > > > > On Thu, Dec 5, 2024 at 2:21 AM VGalaxies <vgalax...@apache.org> wrote: > > > > > > > > > > Hello Incubator Community, > > > > > > > > > > This is a call for a vote to release Apache HugeGraph (Incubating) > > > > > version 1.5.0-rc3 > > > > > > > > > > The Apache HugeGraph community has voted on and approved a proposal to > > > > > release Apache HugeGraph(Incubating) version 1.5.0-rc3 > > > > > > > > > > In this release, we fixed a string of problems mentioned by Justin > > > > > Mclean: > > > > > 1. Exclude the style directory from the hugegraph-ai source package > > > > > and add proper ASF headers to its files. > > > > > 2. Fix the incorrect LICENSE file from the hugegraph-ai source > > > > > package. > > > > > 3. Upgrade the org.json:json dependency in the main repository to a > > > > > version under the "Public Domain" license, ensuring ALv2 compatibility > > > > > (we confirm that this dependency is only for test scope, so there is > > > > > no need to declare it in binary LICENSE). > > > > > > > > > > We now kindly request the Incubator PMC members review and vote on > > > > > this incubator release. > > > > > > > > > > HugeGraph community vote thread: > > > > > • https://lists.apache.org/thread/jfcr2vlrwn7bzmg32d8ljqpmc6794vj0 > > > > > > > > > > Vote result thread: > > > > > • https://lists.apache.org/thread/mrywryb99l7n75g7voqp8n0bk7wnlx5k > > > > > > > > > > The release candidate: > > > > > • https://dist.apache.org/repos/dist/dev/incubator/hugegraph/1.5.0 > > > > > > > > > > Git tag & Commit hash for the release: > > > > > • https://github.com/apache/incubator-hugegraph/tree/1.5.0-rc3 > > > > > (8b90977) > > > > > • > > > > > https://github.com/apache/incubator-hugegraph-toolchain/tree/1.5.0-rc3 > > > > > (4b89c5d) > > > > > • > > > > > https://github.com/apache/incubator-hugegraph-computer/tree/1.5.0-rc3 > > > > > (1a0352e) > > > > > • https://github.com/apache/incubator-hugegraph-ai/tree/1.5.0-rc3 > > > > > (ef56263) > > > > > > > > > > Keys to verify the Release Candidate: > > > > > • https://downloads.apache.org/incubator/hugegraph/KEYS > > > > > > > > > > The release GPG user ID: VGalaxies <vgalax...@apache.org> > > > > > > > > > > The vote will be open for at least 72 hours or until the necessary > > > > > number of votes are reached. > > > > > > > > > > Please vote accordingly: > > > > > [ ] +1 approve > > > > > [ ] +0 no opinion > > > > > [ ] -1 disapprove with the reason > > > > > > > > > > More detail checklist please refer: > > > > > • > > > > > https://cwiki.apache.org/confluence/display/INCUBATOR/Incubator+Release+Checklist > > > > > > > > > > Steps to validate the release，Please refer to: > > > > > • > > > > > https://hugegraph.apache.org/docs/contribution-guidelines/validate-release/ > > > > > (EN) > > > > > • > > > > > https://hugegraph.apache.org/cn/docs/contribution-guidelines/validate-release/ > > > > > (CN) > > > > > > > > > > > > > > > Thanks, > > > > > On behalf of Apache HugeGraph (Incubating) community > > > > > > > > > > --------------------------------------------------------------------- > > > > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > > > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
