Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/567
  
    @suyogmapara Thanks for adding the documentation, it's very useful. I think 
it was a very good idea to put it next to `snapCount`. There's one thing 
however which caught my eye from snapCount doc:
    
    > In order to prevent all of the machines in the quorum from taking a 
snapshot at the same time, each ZooKeeper server will take a snapshot when the 
number of transactions in the transaction log reaches a runtime generated 
random value in the [snapCount/2+1, snapCount] range.The default snapCount is 
100,000.
    
    Isn't `txnLogSizeLimitInKb` affected by the same issue? Do you think it 
would be useful to implement something similar for the new option as well?


---

Reply via email to