[ 
https://issues.apache.org/jira/browse/CASSANDRA-18262?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17698976#comment-17698976
 ] 

Ekaterina Dimitrova commented on CASSANDRA-18262:
-------------------------------------------------

{quote}I think we should see how CASSANDRA-18239 shakes out with regard to the 
eclipse warnings, since those will not currently work in j11.
{quote}
Agreed
{quote}Checkstyle is a completely different story, and Just Works under not 
only j11, but also j17.
{quote}
I haven't tested with 17 but I assume from this comment you tested and you know 
that checkstyle supports JDK17 already?

I would like to propose to exclude eclipse-warnings from this ticket scope in 
favor of CASSANDRA-18239. If that one is not going to land soon - we can open 
separate one for fixing eclipse-warnings when CASSANDRA-18239 gets more clarity 
on next actions and timelines. 

I thought we want to switch the targets to be run in CI only with 11 now, the 
way they were run only with 8 before but with this patch they run with both 8 
and 11. I think the choice of running those targets with only one of the JDK 
versions was done in favor of saving resources. 

 

> Switch ant targets running only with JDK8 to be run with JDK11
> --------------------------------------------------------------
>
>                 Key: CASSANDRA-18262
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18262
>             Project: Cassandra
>          Issue Type: Task
>          Components: Build
>            Reporter: Ekaterina Dimitrova
>            Assignee: Brandon Williams
>            Priority: Normal
>             Fix For: 5.x
>
>
> Currently we run checkstyle, eclipse warnings (and probably anything else?) 
> only with JDK8.
> In preparation to remove JDK8 we need to switch those to run with JDK11.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to