Skip to content

Commit be50629

Browse files
committed
fix(clippy): use sort_by_key instead of sort_by for reverse sorting
- Fix cortex-apply-patch backup.rs: use sort_by_key with Reverse - Fix cortex-compact log_pruner.rs: use sort_by_key with Reverse These changes address clippy::unnecessary_sort_by warnings.
1 parent 454619f commit be50629

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/cortex-apply-patch/src/backup.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ impl BackupManager {
232232
}
233233

234234
// Sort by timestamp (newest first)
235-
backups.sort_by(|a, b| b.timestamp.cmp(&a.timestamp));
235+
backups.sort_by_key(|b| std::cmp::Reverse(b.timestamp));
236236

237237
Ok(backups)
238238
}

src/cortex-compact/src/auto_compaction/log_pruner.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ impl LogPruner {
9696
}
9797

9898
// Sort by modified time (newest first) for rotation priority
99-
log_files.sort_by(|a, b| b.modified.cmp(&a.modified));
99+
log_files.sort_by_key(|f| std::cmp::Reverse(f.modified));
100100

101101
// Process each log file
102102
for (idx, log_file) in log_files.iter().enumerate() {

0 commit comments

Comments
 (0)