search

vert-x3 / vertx-zookeeper

Zookeeper based cluster manager implementation
Other
73 stars 67 forks source link

Broken TCK test #115

Closed pmlopes closed 3 years ago

pmlopes commented 3 years ago

Currently CI is failing due to a "TCK" test that never completes.

I've disable the test for now:

https://github.com/vert-x3/vertx-zookeeper/commit/02cfdbd3dfec0cb2db8b9cebf51494acc6ad7d69#diff-d6640b3a82a3c8a932e251dd24d1bd07410a7d57f141af58e690dbeb984f27c9

As in the disable comment, it seems that the zookeeper behavior is not showing the expectation:

  1. get the async map: foo
  2. store the value "molo" under the key "pipo"
  3. store the value "mili" under the key "pipo" with TTL 15
  4. get the async map: foo
  5. every 15, check if key "pipo" is NULL <-- THIS NEVER HAPPENS
pmlopes commented 3 years ago

Apparently on CI due to the constrained environment the same is happening on the session store test with retry.

tsegismont commented 3 years ago

This is a limitation of the AsyncMap emulation on top of Zookeeper.