Skip to content

Commit

Permalink
Remove debug message about pruning active leaves (#6440)
Browse files Browse the repository at this point in the history
# Description

The debug message was added to identify a potential memory leak.
However, recent observations show that pruning works as expected.
Therefore, it is best to remove this line, as it generates quite
annoying logs.


## Integration

Doesn't affect downstream projects.

---------

Co-authored-by: GitHub Action <[email protected]>
  • Loading branch information
AndreiEres and actions-user authored Nov 13, 2024
1 parent 0156ca8 commit 5aeaa66
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
2 changes: 0 additions & 2 deletions polkadot/node/core/pvf/src/execute/queue.rs
Original file line number Diff line number Diff line change
Expand Up @@ -305,8 +305,6 @@ impl Queue {
for hash in &update.deactivated {
let _ = self.active_leaves.remove(&hash);
}

gum::debug!(target: LOG_TARGET, size = ?self.active_leaves.len(), "Active leaves pruned");
}

fn insert_active_leaf(&mut self, update: ActiveLeavesUpdate, ancestors: Vec<Hash>) {
Expand Down
8 changes: 8 additions & 0 deletions prdoc/pr_6440.prdoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
title: Remove debug message about pruning active leaves
doc:
- audience: Node Dev
description: |-
Removed useless debug message
crates:
- name: polkadot-node-core-pvf
validate: false

0 comments on commit 5aeaa66

Please sign in to comment.