From: Yan, Zheng <zyan@redhat.com>
Date: Wed, 22 Nov 2017 04:33:09 +0000 (+0800)
Subject: mds: remove useless check in Migrator::export_dir
X-Git-Tag: v12.2.3~143^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F19360%2Fhead;p=ceph.git

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" <zyan@redhat.com>
(cherry picked from commit 7afe92f644c97ae1282837bc75cde0b4a8054d0f)
---

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)