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

Ekaterina Dimitrova commented on CASSANDRA-17909:
-------------------------------------------------

The PR was approved on GH. Blocking final rebase, CI run and commit on 
CASSANDRA-18255

> Clean SyncUtil from dead code and update it for new JDK versions
> ----------------------------------------------------------------
>
>                 Key: CASSANDRA-17909
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17909
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Local/Other
>            Reporter: Ekaterina Dimitrova
>            Assignee: Ekaterina Dimitrova
>            Priority: Normal
>             Fix For: 5.x
>
>
> During code inspection I noticed [dead 
> code|https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/utils/SyncUtil.java#L77-L87]
>  (JDK 7) in SyncUtil.
> From a very quick skim _I think_ the [Java 8 
> section|https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/utils/SyncUtil.java#L65-L75]
>  is applicable in JDK 11, not sure for JDK 17 but it seems it should stay at 
> least until we have JDK11. To be revisited. 



--
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