[ https://issues.apache.org/jira/browse/CASSANDRA-17909?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17738831#comment-17738831 ]
Ekaterina Dimitrova commented on CASSANDRA-17909: ------------------------------------------------- CI results seem ok to me. Outstanding failures that do not match what we see with JDK17 are below, and they are unrelated to this ticket: * testTimeout [failed in the JDK17 run,|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2399/workflows/db9c862b-bf3b-4f5d-928b-530838b05e3c/jobs/28006/tests] but It doesn't seem related to what we do here. Running it in a loop on trunk without this patch proved it rare and flaky. I opened ticket CASSANDRA-18641 for it. * testPagingWithClustering-.jdk17 is a known failure from CASSANDRA-18432 > 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: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org