From ee06a16ee8c8434f75adf05c04f94607a5ec090f Mon Sep 17 00:00:00 2001 From: "Yan, Zheng" Date: Wed, 22 Nov 2017 12:33:09 +0800 Subject: [PATCH] mds: remove useless check in Migrator::export_dir There is an assert(dest != mds->get_nodeid()) at very beginning of the function. There also is a check for if 'dest' mds is active. Signed-off-by: "Yan, Zheng" (cherry picked from commit 7afe92f644c97ae1282837bc75cde0b4a8054d0f) --- src/mds/Migrator.cc | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/mds/Migrator.cc b/src/mds/Migrator.cc index 41fe12795139..8649d337786c 100644 --- a/src/mds/Migrator.cc +++ b/src/mds/Migrator.cc @@ -820,10 +820,6 @@ void Migrator::export_dir(CDir *dir, mds_rank_t dest) dout(7) << "already exporting" << dendl; return; } - if (dest == mds->get_nodeid() || !mds->mdsmap->is_up(dest)) { - dout(7) << "cannot export: dest " << dest << " is me or is not active" << dendl; - return; - } if (g_conf->mds_thrash_exports) { // create random subtree bound (which will not be exported) -- 2.47.3