fix: use spawn_blocking fsync

keep the wasteful double-fsync while we have unsolved issues likely moving the fsyncs around.
neondatabase · Nov 10, 2023 · f001d70 · f001d70
1 parent 4eba16d
commit f001d70
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/pageserver/src/tenant/timeline.rs b/pageserver/src/tenant/timeline.rs
@@ -3506,11 +3506,15 @@ impl Timeline {
             // minimize latency.
             //
             // FIXME: spawn_blocking above for this
-            par_fsync::par_fsync(&layer_paths).context("fsync all new layers")?;
+            par_fsync::par_fsync_async(&layer_paths)
+                .await
+                .context("fsync all new layers")?;
 
-            par_fsync::par_fsync(&[self.conf.timeline_path(&self.tenant_id, &self.timeline_id)])
-                .context("fsync of timeline dir")?;
+            let timeline_dir = self.conf.timeline_path(&self.tenant_id, &self.timeline_id);
 
+            par_fsync::par_fsync_async(&[timeline_dir])
+                .await
+                .context("fsync of timeline dir")?;
         }
 
         stats.write_layer_files_micros = stats.read_lock_drop_micros.till_now();