kezhuw commented on PR #1950: URL: https://github.com/apache/zookeeper/pull/1950#issuecomment-1544436457
Yesterday, I have a short out of band (eg. not mailing list) discussion with @tisonkun in a group chat about "why etcd forbid duplicated keys in txn"(https://github.com/etcd-io/etcd/pull/4363, https://github.com/etcd-io/etcd/pull/4376). I think it is partially because of "etcd may not want to deliver multiple mutations on same key with same revision". Then I recalled ZooKeeper side. I wrote a test in https://github.com/kezhuw/zookeeper-client-rust/commit/f6cdacbe4132503b9238ab153c5b845c8ffe369f which mutates one key multiple times and asserts multiple watched events. Sadly, it passed. This basically means that 'zxid' by itself can't not distinguish order of multiple mutations for key in multi (though ordered delivery of watched events are still guaranteed). I guess it make this pr less meaningful. We probably should solve this "multiple mutations for one key in multi op" before continue this pr. But I am afraid of it might be a "won't fix". Anyway, I will open a "Wish" jira later. cc @tisonkun @eolivelli -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
