You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ad...@apache.org on 2017/01/10 20:34:50 UTC

kudu git commit: KUDU-1821. Noisy warning from catalog manager

Repository: kudu
Updated Branches:
  refs/heads/master 1ceaa7b2b -> 217dfe475


KUDU-1821. Noisy warning from catalog manager

This commit changes the LOG if the catalog manager is trying to start.
Add additional if statements for this case to show more details.

Change-Id: I6a9b8577e4d857c63df6e66c10d6cf20270cd41e
Reviewed-on: http://gerrit.cloudera.org:8080/5637
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <to...@apache.org>
Reviewed-by: Adar Dembo <ad...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/217dfe47
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/217dfe47
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/217dfe47

Branch: refs/heads/master
Commit: 217dfe475a4745a017cc9946455ffa01c95e1cad
Parents: 1ceaa7b
Author: Jun He <ju...@gmail.com>
Authored: Thu Jan 5 22:22:09 2017 -0800
Committer: Adar Dembo <ad...@cloudera.com>
Committed: Tue Jan 10 20:34:18 2017 +0000

----------------------------------------------------------------------
 src/kudu/master/catalog_manager.cc | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/217dfe47/src/kudu/master/catalog_manager.cc
----------------------------------------------------------------------
diff --git a/src/kudu/master/catalog_manager.cc b/src/kudu/master/catalog_manager.cc
index d2ea3c6..1afd3bb 100644
--- a/src/kudu/master/catalog_manager.cc
+++ b/src/kudu/master/catalog_manager.cc
@@ -389,8 +389,13 @@ void CatalogManagerBgTasks::Run() {
     {
       CatalogManager::ScopedLeaderSharedLock l(catalog_manager_);
       if (!l.catalog_status().ok()) {
-        LOG(WARNING) << "Catalog manager background task thread going to sleep: "
-                     << l.catalog_status().ToString();
+        if (l.catalog_status().IsServiceUnavailable()) {
+          LOG(INFO) << "Waiting for catalog manager background task thread to start: "
+                    << l.catalog_status().ToString();
+        } else {
+          LOG(WARNING) << "Catalog manager background task thread going to sleep: "
+                       << l.catalog_status().ToString();
+        }
       } else if (l.leader_status().ok()) {
         std::vector<scoped_refptr<TabletInfo>> to_process;