Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/453
@revans2 Your latest change looks good to me and a bit safer than the
previous one. Would you please consider adding some unit tests to validate the
functionality?
What do you think of porting testTxnAheadSnapInRetainDB() test from your
codebase?
Maybe I can help making it not flaky, if you think it correctly verifies
the original issue.---
