li4wang
li4wang
> When has synchronized been added to take/restore snapshot? synchronized on the ZookeeperServer object was added when we introduced remote backup and restore feature, specifically https://github.com/apache/zookeeper/pull/1943 > Are you saying...
Thanks @anmolnar and @kezhuw > > I saw that sync is called only in one thread and pendingSyncs is concurrent safe. > I think we could remove synchronize from sync()...
@anmolnar @kezhuw The PR is good for merge now, right? Thanks.