Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/420
Tests don't seem to be related to the snapCount apart from the 2 that
@afine mentioned previously. This way we can keep the structure of this
testfile nice and clean with doing the initialization in the setUp method and
reusing everything from the base class. I prefer to keep it this way.
Regarding making it volatile: ability to change the snapCount is for
testing-purposes only. Which is already not the nicest thing in the world, I
wouldn't make it even worse by making it thread safe just for test code.
Commented lines have been removed, were there by accident.
---
