Skip to content

Commit

Permalink
fix: remove needless move
Browse files Browse the repository at this point in the history
  • Loading branch information
koivunej committed Aug 25, 2023
1 parent b45df96 commit ecf85de
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pageserver/src/tenant/timeline.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1676,7 +1676,7 @@ impl Timeline {
Err(FutureLayer { local }) => {
if local.is_some() {
path.push(name.file_name());
init::cleanup_future_layer(&path, name, disk_consistent_lsn)?;
init::cleanup_future_layer(&path, &name, disk_consistent_lsn)?;
path.pop();
}
needs_cleanup.push(name);
Expand Down
2 changes: 1 addition & 1 deletion pageserver/src/tenant/timeline/init.rs
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ pub(super) fn cleanup_local_file_for_remote(

pub(super) fn cleanup_future_layer(
path: &Path,
name: LayerFileName,
name: &LayerFileName,
disk_consistent_lsn: Lsn,
) -> anyhow::Result<()> {
use LayerFileName::*;
Expand Down

0 comments on commit ecf85de

Please sign in to comment.