zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From suyogmapara <...@git.apache.org>
Subject [GitHub] zookeeper issue #567: ZOOKEEPER-3071: Add a config parameter to control tran...
Date Fri, 10 Aug 2018 17:56:45 GMT
Github user suyogmapara commented on the issue:

    https://github.com/apache/zookeeper/pull/567
  
    Thanks @anmolnar for the review. Regarding randomization, I think they did it for snapCount
because snapshot is potentially expensive IO and entire ensemble doing it at the same time
may be an issue. This feature doesn't do snapshot when we roll the log and just switches the
file we are writing to. So it may be okay if everyone do it at the same time. Please let me
know if you disagree.


---

Mime
View raw message