Release temp full sync snapshot before renaming. (#1369)

* Release temp full sync snapshot before renaming.

Otherwise, it will report `PermissionDenied` on Windows.

* Fix.

core/src/sync/state/restore.rs

@@ -64,8 +64,14 @@ impl Restorer {
 
     /// Start to restore chunks asynchronously.
     pub fn finalize_restoration(
-        &self, state_manager: Arc<StateManager>, snapshot_info: SnapshotInfo,
-    ) -> StorageResult<()> {
+        &mut self, state_manager: Arc<StateManager>,
+        snapshot_info: SnapshotInfo,
+    ) -> StorageResult<()>
+    {
+        // Release temp snapshot db so it can be renamed on Windows.
+        // `self.verifier` is never unwrapped, so it's safe to set it to None,
+        self.verifier = None;
+
         state_manager
             .get_storage_manager()
             .get_snapshot_manager()

core/src/sync/state/snapshot_chunk_sync.rs

@@ -526,10 +526,11 @@ impl SnapshotChunkSync {
                 download_start_time.elapsed()
             );
 
+            let snapshot_info = inner.snapshot_info.clone();
             // start to restore and update status
             inner.restorer.finalize_restoration(
                 ctx.manager.graph.data_man.storage_manager.clone(),
-                inner.snapshot_info.clone(),
+                snapshot_info,
             )?;
             inner.status = Status::Completed;
         }