Github user anmolnar commented on the issue:
    @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 
    What do you think of porting testTxnAheadSnapInRetainDB() test from your 
    Maybe I can help making it not flaky, if you think it correctly verifies 
the original issue.


Reply via email to