[CASSANDRA-17909] Clean SyncUtil from dead code and update it for new JDK versions

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 5.0-alpha1, 5.0
• Component/s: Local/Other
• Epic Link: Support Java 17
• Bug Category: Correctness
• Severity: Normal
• Complexity: Normal
• Discovered By: User Report
• Platform: All

Source Control Link

https://github.com/apache/cassandra/commit/b92184e90cfee834b1fcb0866a3f80959b9455de

Test and Documentation Plan

Notable changes:

• old Java 7 and 8 code was removed. This effectively makes this patch commit blocked on CASSANDRA-18255
• We should not be throwing any more UnsupportedOperationException here. This comment is not applicable anymore since this fix fix in JDK11.

trunk PR, CI
I can see only the known failures from the table in CASSANDRA-16895 in CI.

• Assignee: Ekaterina Dimitrova
• Reporter: Ekaterina Dimitrova

Authors

Ekaterina Dimitrova

Reviewers

Jacek Lewandowski

• Created: 21/Sep/22 03:02
• Updated: 12/Sep/23 13:02
• Resolved: 25/Jul/23 19:45

Description

During code inspection I noticed dead code (JDK 7) in SyncUtil.

From a very quick skim I think the Java 8 section 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. 

Issue Links

• is blocked by: CASSANDRA-18255 Drop JDK8 Support