From b4c81f7dff30fb31bd5c6d2976c1589ace6a1b31 Mon Sep 17 00:00:00 2001 From: Joonas Koivunen Date: Tue, 29 Aug 2023 14:13:09 +0300 Subject: [PATCH] refactor: rename LayerInner::on_drop to on_downloaded_layer_drop --- pageserver/src/tenant/storage_layer/layer.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pageserver/src/tenant/storage_layer/layer.rs b/pageserver/src/tenant/storage_layer/layer.rs index bf33a941b6..dbb2c576eb 100644 --- a/pageserver/src/tenant/storage_layer/layer.rs +++ b/pageserver/src/tenant/storage_layer/layer.rs @@ -816,7 +816,7 @@ impl LayerInner { } /// `DownloadedLayer` is being dropped, so it calls this method. - fn on_drop(self: Arc) { + fn on_downloaded_layer_drop(self: Arc) { let gc = self.wanted_garbage_collected.load(Ordering::Acquire); let evict = self.wanted_evicted.load(Ordering::Acquire); let can_evict = self.have_remote_client; @@ -993,7 +993,7 @@ impl std::fmt::Debug for DownloadedLayer { impl Drop for DownloadedLayer { fn drop(&mut self) { if let Some(owner) = self.owner.upgrade() { - owner.on_drop(); + owner.on_downloaded_layer_drop(); } else { // no need to do anything, we are shutting down }