search

ksprojects / zkcopy

ZooKeeper copy utililty
Apache License 2.0
237 stars 94 forks source link

fix::removeDeprecated exclude 'zookeeper' node #25

Closed tonycody closed 10 months ago

tonycody commented 11 months ago

Delete node data exclusion zookeeper path when A zookeeper is fully synchronized to B zookeeper

kshchepanovskyi commented 10 months ago

Hello, thank you for your change, but this issue is already addressed in this commit (it was merged recently: https://github.com/ksprojects/zkcopy/commit/7417f9390cdd8ff6ec60227d6980dca55a2d9cdf)

In your change, as far as I understand, the zookeeper path will be excluded from any path. But reserved is only /zookeeper when it is directly under the root.