Github user yisong-yue commented on the issue:
https://github.com/apache/zookeeper/pull/690
@anmolnar
One drawback I see with this structure is that it makes it harder to
dynamically pick deserialization mode based on a snapshot's filename, since
`FileSnap` manages a whole `snapDir` directory now. Maybe we should separate
directory management logic from snapshot file (de)serialization in FileSnap
class.---
