Thanks Ekaterina! I think the security manager ticket is CASSANDRA-18711 (correct me if I’m wrong) - sorry missed this in my previous search.
> 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. No objections from me since these issues are mostly cosmetic, but it would be nice to clear these before the next alpha/beta. I will create a ticket for the unknown module warning later if nobody beats me to it. On Fri, 3 Nov 2023 at 21:17 Ekaterina Dimitrova <e.dimitr...@gmail.com> wrote: > 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 >>>>> >>>>
