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

André Filipe Aloise commented on GROOVY-9852:
---------------------------------------------

Vert.x 4.0 GA uses Groovy 3.0.7 as well as Vert.x 4.0.0-SNAPSHOT. Unfortunately 
the bug related here is present.

 

Vert.x 3.9.5-SNAPSHOT also uses Groovy 3.0.7, and it works fine.

 

This seems to be a different bug from GROOVY-9815, as it started to happen with 
Vert.x 4.0.0.Beta2 which has Groovy 3.0.5.

 

In Vert.x 4.0 branch, there was a design change regarding Promise and Future. 
This is why GROOVY-9815 was affecting only Vert.x 3.9 branch.

 

So to sum up. Vert.x 3.9 had an issue with interface method selection, that 
didn't happen to Vert.x 4.0 due to a code chance. GROOVY-9815 fixed the problem 
with Vert.x 3.9. But now, we are facing a behaviou that doesn't manifest in 3.9 
but only in 4.0, starting with Beta2 which uses Groovy 3.0.5, and even using 
Groovy 3.0.7 the error persists.

> Result is already complete (IllegalStateException)
> --------------------------------------------------
>
>                 Key: GROOVY-9852
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9852
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 3.0.5
>         Environment: Windows 10
> OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_272-b10)
>            Reporter: André Filipe Aloise
>            Priority: Major
>
> Steps:
> 1 - Consider this reproducer [https://github.com/ptomaszek/vertx-groovy]
> 2 - Edit build.gradle and change vertxVersion to '4.0.0.Beta1' (uses Groovy 
> 3.0.3)
> 3 - Edit org.example.MainGroovy and replace the code of main method with the 
> following:
> {code:java}
> Promise p = Promise.promise()
> p.future().onSuccess { println it }
> p.complete("Done with Groovy!!!"){code}
> The code runs fine.
> But, if in Step 2 the vertxVersion in build.gradle is changed to 
> '4.0.0.Beta2' (uses Groovy 3.0.5), the code from Step 3 will fail with an 
> IllegalStateException. 
> {code:java}
> Exception in thread "main" java.lang.IllegalStateException: Result is already 
> complete
>  at io.vertx.core.Promise.complete(Promise.java:67)
>  at io.vertx.core.Promise$complete$1.call(Unknown Source)
>  at 
> org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:47)
>  at 
> org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:125)
>  at 
> org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:139)
>  at org.example.MainGroovy.main(MainGroovy.groovy:17)
> {code}
> If you use @CompileStatic on main method with Step 3, with Beta2, it works 
> fine. The @CompileStatic is not necessary to run well with Vert.x 
> '4.0.0.Beta1'. A Java class with similar code will work using any version of 
> Vert.x (Beta1, Beta2, Beta3, CR1....), so doesn't looks like a Java or Vertx 
> issue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to