cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jason Brown (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-9608) Support Java 11
Date Fri, 29 Jun 2018 19:10:00 GMT


Jason Brown commented on CASSANDRA-9608:

I've taken a first pass through the scripts and build.xml parts (all the non-code stuff),
and on the whole it's looking pretty good. I've made a few minor comments on the PR, but I
have these points, as well:
 - you have the java version check code copied across several scripts, and it looks like
is used by many other scripts (so I guess this is the 'canonical' location?). Should we have
the main bin/ use this? Maybe the one in tools/bin, as well? Also, I don't
think (I may be wrong) the in the debian/redhat can call the
script. wdyt?
 - the changes you made to need to be made to cassandra-env.ps1 (wrt java
version checking). There's also some Windows {{.bat}} files in the code base. Can you check
to see if they need updates, as well?
 - can we add a simple note to conf/jvm8-clients.options, with something like this: "this
file is intentionaly blank". or should we just get rid of it, and add it when we actually
need it?

I suspect the code will be easier to review, so hopefully I can knock that out in short order.

> Support Java 11
> ---------------
>                 Key: CASSANDRA-9608
>                 URL:
>             Project: Cassandra
>          Issue Type: Task
>            Reporter: Robert Stupp
>            Assignee: Robert Stupp
>            Priority: Minor
>             Fix For: 4.x
>         Attachments: jdk_9_10.patch
> This ticket is intended to group all issues found to support Java 9 in the future.
> From what I've found out so far:
> * Maven dependency {{com.sun:tools:jar:0}} via cobertura cannot be resolved. It can be
easily solved using this patch:
> {code}
> -        <dependency groupId="net.sourceforge.cobertura" artifactId="cobertura"/>
> +        <dependency groupId="net.sourceforge.cobertura" artifactId="cobertura">
> +          <exclusion groupId="com.sun" artifactId="tools"/>
> +        </dependency>
> {code}
> * Another issue is that {{sun.misc.Unsafe}} no longer contains the methods {{monitorEnter}}
+ {{monitorExit}}. These methods are used by {{o.a.c.utils.concurrent.Locks}} which is only
used by {{o.a.c.db.AtomicBTreeColumns}}.
> I don't mind to start working on this yet since Java 9 is in a too early development

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message