From 244feb08d21d8fa3cd5812e2bd9aa1e03e7333a3 Mon Sep 17 00:00:00 2001 From: Venky Shankar Date: Tue, 29 Nov 2022 17:11:35 +0530 Subject: [PATCH] mds: remove redundant frag check in MDBalancer::maybe_fragment() CDir::should_merge() checks this anyway. Signed-off-by: Venky Shankar --- src/mds/MDBalancer.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/mds/MDBalancer.cc b/src/mds/MDBalancer.cc index bd00db9f53f1..baa43bb43412 100644 --- a/src/mds/MDBalancer.cc +++ b/src/mds/MDBalancer.cc @@ -1243,8 +1243,7 @@ void MDBalancer::maybe_fragment(CDir *dir, bool hot) } // merge? - if (dir->get_frag() != frag_t() && dir->should_merge() && - merge_pending.count(dir->dirfrag()) == 0) { + if (dir->should_merge() && merge_pending.count(dir->dirfrag()) == 0) { queue_merge(dir); } } -- 2.47.3