Put OffloadedTimeline into Arc

neondatabase · Sep 30, 2024 · 1b4212d · 1b4212d
1 parent 386ab88
commit 1b4212d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pageserver/src/tenant.rs b/pageserver/src/tenant.rs
@@ -291,7 +291,7 @@ pub struct Tenant {
     timelines_creating: std::sync::Mutex<HashSet<TimelineId>>,
 
     /// Possibly offloaded and archived timelines
-    timelines_offloaded: Mutex<HashMap<TimelineId, OffloadedTimeline>>,
+    timelines_offloaded: Mutex<HashMap<TimelineId, Arc<OffloadedTimeline>>>,
 
     // This mutex prevents creation of new timelines during GC.
     // Adding yet another mutex (in addition to `timelines`) is needed because holding

diff --git a/pageserver/src/tenant/timeline/offload.rs b/pageserver/src/tenant/timeline/offload.rs
@@ -28,7 +28,7 @@ pub(crate) async fn offload_timeline(
         let mut offloaded_timelines = tenant.timelines_offloaded.lock().unwrap();
         offloaded_timelines.insert(
             timeline.timeline_id,
-            OffloadedTimeline::from_timeline(&timeline),
+            Arc::new(OffloadedTimeline::from_timeline(&timeline)),
         );
     }