Merge branch 'master' into write

baidu · Oct 12, 2016 · 374d83b · 374d83b
2 parents 872b96d + 6335dd2
commit 374d83b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/src/flags.cc b/src/flags.cc
@@ -23,7 +23,7 @@ DEFINE_int32(keepalive_timeout, 10, "Chunkserver keepalive timeout");
 DEFINE_int32(default_replica_num, 3, "Default replica num of data block");
 DEFINE_int32(nameserver_log_level, 4, "Nameserver log level");
 DEFINE_string(nameserver_warninglog, "./wflog", "Warning log file");
-DEFINE_int32(nameserver_start_recover_timeout, 120, "Nameserver starts recover in second");
+DEFINE_int32(nameserver_start_recover_timeout, 3600, "Nameserver starts recover in second");
 DEFINE_int32(recover_speed, 100, "Max num of block to recover for one chunkserver");
 DEFINE_int32(recover_dest_limit, 5, "Number of recover dest");
 DEFINE_int32(hi_recover_timeout, 180, "Recover timeout for high priority blocks");

diff --git a/src/nameserver/nameserver_impl.cc b/src/nameserver/nameserver_impl.cc
@@ -110,7 +110,7 @@ void NameServerImpl::CheckRecoverMode() {
     work_thread_pool_->DelayTask(1000, boost::bind(&NameServerImpl::CheckRecoverMode, this));
 }
 void NameServerImpl::LeaveReadOnly() {
-    LOG(INFO, "Nameserver leave safemode");
+    LOG(INFO, "Nameserver leave read only");
     if (readonly_) {
         readonly_ = false;
     }
@@ -1022,27 +1022,30 @@ bool NameServerImpl::WebService(const sofa::pbrpc::HTTPRequest& request,
         ListRecover(&response);
         return true;
     } else if (path == "/dfs/hi_only") {
+        recover_timeout_ = 0;
         LOG(INFO, "ChangeRecoverMode hi_only");
         recover_mode_ = kHiOnly;
         response.content->Append("<body onload=\"history.back()\"></body>");
         return true;
     } else if (path == "/dfs/recover_all") {
+        recover_timeout_ = 0;
         LOG(INFO, "ChangeRecoverMode recover_all");
         recover_mode_ = kRecoverAll;
         response.content->Append("<body onload=\"history.back()\"></body>");
         return true;
     } else if (path == "/dfs/stop_recover") {
+        recover_timeout_ = 0;
         LOG(INFO, "ChangeRecoverMode stop_recover");
         recover_mode_ = kStopRecover;
         response.content->Append("<body onload=\"history.back()\"></body>");
         return true;
-    } else if (path == "/dfs/entry_read_only") {
-        LOG(INFO, "ChangeStatus entry_read_only");
+    } else if (path == "/dfs/leave_read_only") {
+        LOG(INFO, "ChangeStatus leave_read_only");
         LeaveReadOnly();
         response.content->Append("<body onload=\"history.back()\"></body>");
         return true;
-    } else if (path == "/dfs/leave_read_only") {
-        LOG(INFO, "ChangeStatus leave_read_only");
+    } else if (path == "/dfs/entry_read_only") {
+        LOG(INFO, "ChangeStatus entry_read_only");
         readonly_ = true;
         response.content->Append("<body onload=\"history.back()\"></body>");
         return true;
@@ -1236,7 +1239,7 @@ bool NameServerImpl::WebService(const sofa::pbrpc::HTTPRequest& request,
     str += "<div class=\"col-sm-4 col-md-4\">";
     str += "Total: " + common::HumanReadableString(total_quota) + "B</br>";
     str += "Used: " + common::HumanReadableString(total_data) + "B</br>";
-    str += "Pending tasks: "
+    str += "Pending: (r/w/rp/h)<br>"
         + common::NumToString(read_thread_pool_->PendingNum()) + " "
         + common::NumToString(work_thread_pool_->PendingNum()) + " "
         + common::NumToString(report_thread_pool_->PendingNum()) + " "
@@ -1249,13 +1252,13 @@ bool NameServerImpl::WebService(const sofa::pbrpc::HTTPRequest& request,
     str += "<div class=\"col-sm-4 col-md-4\">";
     str += "Status: ";
     if (readonly_) {
-        str += "<font color=\"red\">Read Only</font></br> <a href=\"/dfs/entry_read_only\">LeaveSafeMode</a>";
+        str += "<font color=\"red\">Read Only</font></br> <a href=\"/dfs/leave_read_only\">LeaveReadOnly</a>";
     } else {
-        str += "Normal</br> <a href=\"/dfs/leave_read_only\">EnterSafeMode</a>";
+        str += "Normal</br> <a href=\"/dfs/entry_read_only\">EnterReadOnly</a>";
     }
     str += "</br>";
     if (recover_timeout_ > 1) {
-        str += "RecoverCountdown: " + common::NumToString(recover_timeout_) + "</br>";
+        str += "Recover: " + common::NumToString(recover_timeout_) + "<a href=\"/dfs/stop_recover\"> Stop</a></br>";
     }
     str += "RecoverMode: ";
     if (recover_mode_ == kRecoverAll) {