Address whitewum's comments

vesoft-inc · Jul 12, 2019 · f92944b · f92944b
1 parent 6db8f35
commit f92944b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/meta/processors/admin/AdminClient.cpp b/src/meta/processors/admin/AdminClient.cpp
@@ -329,6 +329,7 @@ StatusOr<std::vector<HostAddr>> AdminClient::getPeers(GraphSpaceID spaceId, Part
         case kvstore::ResultCode::ERR_KEY_NOT_FOUND:
             return Status::Error("Key Not Found");
         default:
+            LOG(WARNING) << "Get peers failed, error " << static_cast<int32_t>(code);
             break;
     }
     return Status::Error("Get Failed");

diff --git a/src/meta/processors/admin/BalanceTask.cpp b/src/meta/processors/admin/BalanceTask.cpp
@@ -23,7 +23,7 @@ const std::string kBalanceTaskTable = "__b_task__"; // NOLINT
 void BalanceTask::invoke() {
     CHECK_NOTNULL(client_);
     if (ret_ == Result::FAILED) {
-        endTimeMs_ = time::WallClock::fastNowInSec();
+        endTimeMs_ = time::WallClock::fastNowInMilliSec();
         saveInStore();
         onError_();
         return;
@@ -33,7 +33,7 @@ void BalanceTask::invoke() {
             LOG(INFO) << taskIdStr_ << "Start to move part!";
             status_ = Status::CHANGE_LEADER;
             ret_ = Result::IN_PROGRESS;
-            startTimeMs_ = time::WallClock::fastNowInSec();
+            startTimeMs_ = time::WallClock::fastNowInMilliSec();
         }
         case Status::CHANGE_LEADER: {
             LOG(INFO) << taskIdStr_ << "Ask the src to give up the leadership.";