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

Ekaterina Dimitrova edited comment on CASSANDRA-18395 at 4/10/23 6:04 PM:
--------------------------------------------------------------------------

I can confirm that current trunk fails with JDK19 and when the patch is applied 
I can successfully run:
{code:java}
ant jar build-test{code}
I agree with the provided explanation. Patch posted here 

[https://github.com/ekaterinadimitrova2/cassandra/pull/new/CASSANDRA-18395]

CI pushed 
[here|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra?branch=CASSANDRA-18395]
I will check the CI results later


was (Author: e.dimitrova):
I can confirm that current trunk fails with JDK19 and when the patch is applied 
I can successfully run
{code:java}
ant jar build-test{code}
Patch posted here 

https://github.com/ekaterinadimitrova2/cassandra/pull/new/CASSANDRA-18395

CI pushed 
[here|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra?branch=CASSANDRA-18395]
I will check the CI results later

> Rename internal state() method in AbstractFuture to not conflict with Java 19 
> changes
> -------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-18395
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18395
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Build
>            Reporter: Sylwester Lachiewicz
>            Assignee: Sylwester Lachiewicz
>            Priority: Normal
>             Fix For: 5.0
>
>
> From Java 19 we have new method Future.state and it conflicts with our 
> private method. Fix bellow. 
> [https://download.java.net/java/early_access/jdk20/docs/api/java.base/java/util/concurrent/Future.html#state()]
>  
> {code:java}
> +++ b/src/java/org/apache/cassandra/utils/concurrent/AbstractFuture.java
> @@ -494,11 +494,11 @@ public abstract class AbstractFuture<V> implements 
> Future<V>
>      public String toString()
>      {
>          String description = description();
> -        String state = state();
> +        String state = stateInfo();
>          return description == null ? state : (state + ' ' + description);
>      }
>  
> -    private String state()
> +    private String stateInfo()
>      {
>          Object result = this.result;
>          if (isSuccess(result)){code}



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

Reply via email to