rust/neon
1
0
Fork 0
mirror of https://github.com/neondatabase/neon.git synced 2026-05-31 12:00:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: minor cleanup, doc

Browse Source
This commit is contained in:
Joonas Koivunen
2023-08-24 22:53:32 +03:00
parent 2ecf6727c5
commit 6ccc6cbc69
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
pageserver/src/tenant/storage_layer/layer.rs
View File

@@ -966,6 +966,7 @@ impl std::fmt::Debug for ResidentLayer {
}
impl ResidentLayer {
/// Release the eviction guard, making this back into a plain [`Layer`].
pub(crate) fn drop_eviction_guard(self) -> Layer {
self.into()
}
@@ -977,11 +978,11 @@ impl ResidentLayer {
) -> anyhow::Result<Vec<DeltaEntry<'_>>> {
use LayerKind::*;
let inner = &self.owner.0;
let owner = &self.owner.0;
match self.downloaded.get(inner).await? {
match self.downloaded.get(owner).await? {
Delta(d) => {
inner
owner
.access_stats
.record_access(LayerAccessKind::KeyIter, ctx);
@@ -1053,10 +1054,10 @@ impl Drop for DownloadedLayer {
}
impl DownloadedLayer {
/// The owner is required so that we don't have to upgrade the `Self::owner`, which will only
/// be used on drop. This way, initializing a DownloadedLayer with the owner gone is
/// impossible.
async fn get(&self, owner: &LayerInner) -> anyhow::Result<&LayerKind> {
// the owner is required so that we don't have to upgrade the self.owner, which will only
// be used on drop. this way, initializing a DownloadedLayer without an owner is statically
// impossible, so we can just not worry about it.
let init = || async {
// there is nothing async here, but it should be async
if owner.desc.is_delta {