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

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

I think I see great ideas here but I believe they are off-topic for this 
ticket. Shall we open another one/ones and discussion thread to explore them?

Now about the checkstyle, I also realized that we make it harder for people who 
might try just to run in their environment with non-LTS JDK versions for 
testing. They can try to build with 
ant -Djava.supported=13 jar
where 13 can be substituted with other version, it is just for the sake of the 
example. 

We saw how JDK 8 and newer versions required different checkstyle versions. I 
guess people can hit a need of different checkstyle version between different 
non-LTS versions which we do not actively monitor and we do not plan to add in 
CI. With the currently committed patch we run checkstyle even with those 
non-LTS versions. At the same time we see people proactively looking into 
Cassandra so we can make hopefully quicker and easier the next upgrades when 
the next LTS version lands (example - CASSANDRA-18395)

> Switch checkstyle 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: Maxim Muzafarov
>            Priority: Normal
>             Fix For: 5.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> 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