vert-x3 / vertx-ignite

Apache License 2.0
35 stars 28 forks source link

Prevent race during node listener registration #110

Closed zyclonite closed 3 years ago

zyclonite commented 3 years ago

Motivation:

There are some edge cases were a nodeListener registration can fail (different threads accessing the cluster manager) Synchronizing this method solves that issue

vietj commented 3 years ago

it might not be a race but a visibility issue instead, using synchronized enforces this

zyclonite commented 3 years ago

true, like volatility