Expose AsyncMap keys, values and entries

tsegismont commented 7 years ago

tsegismont commented 7 years ago

@stream1984 can you please review? But please do not merge yet since @vietj must approve the PR in core first. Thanks

stream-iori commented 7 years ago

Got it.

tsegismont commented 7 years ago

@stream1984 a few things to note:

this PR changes the ways keys are stored otherwise we can't recreate the original object.

As a consequence:

3.5 nodes will not be able to connect to nodes with an earlier version
There is a new restriction on key types: before it could be any object (just a toString call), now it must be either a ClusterSerializable or a Serializable

the Base64 url decoder is used on purpose because it uses - and _ instead of + and / (Zookeeper is confused if a key contains a slash)

vert-x3 / vertx-zookeeper