I am sorry, I totally forgot to address your other concern, Paulo. The security manager is marked for deprecation in JDK 17. So this warning is to stress to people they need to take care of a replacement, sooner than later. I believe we have somewhere an unassigned ticket in Open status to address this topic.
On Fri, 3 Nov 2023 at 21:07, Ekaterina Dimitrova <e.dimitr...@gmail.com> wrote: > Hi Paulo, > > Thank you for testing and for raising the issue! > I can confirm I do not use the same JDK as you, and I did not see any > warnings on my machine on startup or when calling nodetool commands. > > I believe on a quick check that jdk.attach was needed for nodetool sjk. > (It was mentioned on CASSANDRA-16895 at least) > About jdk.compiler - it was added as per this recommendation for chronicle > https://chronicle.software/chronicle-support-java-17/ > > I do not believe we test with the mentioned JDK in CI, so additional > testing will be required to figure out things better. > > As this is alpha release - can we open a ticket to be resolved in the > next alpha/beta? It is up to PMC to decide, of course. Also, we need a bit > more investigation here. I can try to take a look tomorrow in more > detail if no one beats me to that. > > Best regards, > Ekaterina > > > On Fri, 3 Nov 2023 at 20:01, Paulo Motta <pa...@apache.org> wrote: > >> Clarification: >> - When running nodetool only the "Unknown module" warnings show up. All >> warnings show up during startup. >> >> On Fri, Nov 3, 2023 at 7:58 PM Paulo Motta <pa...@apache.org> wrote: >> >>> Launched a tarball-based 5.0-alpha2 container on top of >>> "eclipse-temurin:17-jre-focal" and the server starts up fine, can run >>> nodetool and cqlsh. >>> >>> I got these seemingly harmless JDK17 warnings during startup and when >>> running nodetool (no warnings on JDK11): >>> >>> WARNING: Unknown module: jdk.attach specified to --add-exports >>> WARNING: Unknown module: jdk.compiler specified to --add-exports >>> WARNING: Unknown module: jdk.compiler specified to --add-opens >>> WARNING: A terminally deprecated method in java.lang.System has been >>> called >>> WARNING: System::setSecurityManager has been called by >>> org.apache.cassandra.security.ThreadAwareSecurityManager >>> (file:/opt/cassandra/lib/apache-cassandra-5.0-alpha2-SNAPSHOT.jar) >>> WARNING: Please consider reporting this to the maintainers of >>> org.apache.cassandra.security.ThreadAwareSecurityManager >>> WARNING: System::setSecurityManager will be removed in a future release >>> >>> Anybody knows if these warnings are legit/expected ? We can create >>> follow-up tickets if needed. >>> >>> $ java --version >>> openjdk 17.0.9 2023-10-17 >>> OpenJDK Runtime Environment Temurin-17.0.9+9 (build 17.0.9+9) >>> OpenJDK 64-Bit Server VM Temurin-17.0.9+9 (build 17.0.9+9, mixed mode, >>> sharing) >>> >>> On Fri, Nov 3, 2023 at 6:13 PM Jonathan Ellis <jbel...@gmail.com> wrote: >>> >>>> +1 >>>> >>>> On Mon, Oct 30, 2023 at 3:47 PM Mick Semb Wever <m...@apache.org> wrote: >>>> >>>>> Proposing the test build of Cassandra 5.0-alpha2 for release. >>>>> >>>>> DISCLAIMER, this alpha release does not contain the features: >>>>> Transactional Cluster Metadata (CEP-21) and Accord Transactions >>>>> (CEP-15). These features are under discussion to be pushed to a >>>>> 5.1-alpha1 release, with an eta still this year. >>>>> >>>>> This release does contain Vector Similarity Search (CEP-30). >>>>> >>>>> Please also note that this is an alpha release and what that means, >>>>> further info at >>>>> https://cwiki.apache.org/confluence/display/CASSANDRA/Release+Lifecycle >>>>> >>>>> sha1: ea76d148c374198fede6978422895668857a927f >>>>> Git: https://github.com/apache/cassandra/tree/5.0-alpha2-tentative >>>>> Maven Artifacts: >>>>> >>>>> https://repository.apache.org/content/repositories/orgapachecassandra-1317/org/apache/cassandra/cassandra-all/5.0-alpha2/ >>>>> >>>>> The Source and Build Artifacts, and the Debian and RPM packages and >>>>> repositories, are available here: >>>>> https://dist.apache.org/repos/dist/dev/cassandra/5.0-alpha2/ >>>>> >>>>> The vote will be open for 72 hours (longer if needed). Everyone who >>>>> has tested the build is invited to vote. Votes by PMC members are >>>>> considered binding. A vote passes if there are at least three binding >>>>> +1s and no -1's. >>>>> >>>>> [1]: CHANGES.txt: >>>>> >>>>> https://github.com/apache/cassandra/blob/5.0-alpha2-tentative/CHANGES.txt >>>>> [2]: NEWS.txt: >>>>> https://github.com/apache/cassandra/blob/5.0-alpha2-tentative/NEWS.txt >>>>> >>>> >>>> >>>> -- >>>> Jonathan Ellis >>>> co-founder, http://www.datastax.com >>>> @spyced >>>> >>>
