ztzg commented on PR #1716: URL: https://github.com/apache/zookeeper/pull/1716#issuecomment-2434814408
(@kezhuw: wrote this a few days ago, but it seems GitHub drops email replies now?!) Hi Kezhu, Are you sure this is the issue you want to comment on? "Overflowing" `CloseSessionTxn`'s create problems in *transaction* logs, and I don't see how this would interact with the fuzziness of snapshots. Am I missing something? Cheers, -D P.-S. — I will have a look at `ZOOKEEPER-4874`, but snapshots are not *that* fuzzy given the idempotency of operations. We might want to relitigate, but I would say that the fuzziness was a carefully considered design point in the architecture of ZooKeeper. (Note that in my "own" `ZOOKEEPER-4846`, it's the "ACL storage optimization" which bit us—some new checks were too eager in looking up ACLs before the data structure was fully "reconciled.") -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
