vert-x3 / vertx-ignite

Apache License 2.0
34 stars 28 forks source link

IgniteClusterManager.join may throw java.lang.IllegalStateException: Result is already complete: succeeded #76

Closed emogua closed 5 years ago

emogua commented 5 years ago

fut.complete(); is called twice in line 288 and line 297. It seems like does not effect normal flow. But I'm not sure about that. I think the last call should be removed.

emogua commented 5 years ago

Full stack trace:

8月 09, 2019 5:50:06 下午 io.vertx.core.impl.ContextImpl 严重: Unhandled exception java.lang.IllegalStateException: Result is already complete: succeeded at io.vertx.core.impl.FutureImpl.complete(FutureImpl.java:95) at io.vertx.spi.cluster.ignite.IgniteClusterManager.lambda$null$4(IgniteClusterManager.java:288) at io.vertx.core.impl.ContextImpl.lambda$executeBlocking$2(ContextImpl.java:316) at io.vertx.core.impl.TaskQueue.run(TaskQueue.java:76) at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) at java.base/java.lang.Thread.run(Thread.java:834)

NavidMitchell commented 5 years ago

I am seeing this same error every time I start a cluster node.

emogua commented 5 years ago

I'm afraid this lib is dead, I implement it by myself...

I am seeing this same error every time I start a cluster node.

NavidMitchell commented 5 years ago

@emogua Hopefully they will pull.. :)

vietj commented 5 years ago

@emogua you can become contributor of this project if you want

vietj commented 5 years ago

@NavidMitchell you can become contributor too and make directly improvements.

I merged your PR, thanks

NavidMitchell commented 5 years ago

@vietj I would be happy to become a contributor. Thank You :)

vietj commented 5 years ago

how about starting upgrading Ignite version and having this project running in Travis CI ?

NavidMitchell commented 5 years ago

Yes sure that is something I can work on.