kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject [kudu] branch master updated: [master] log on longer location mapping command runs
Date Fri, 01 Mar 2019 22:11:37 GMT
This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 4d4552d  [master] log on longer location mapping command runs
4d4552d is described below

commit 4d4552d06934f14ed608cf6eb330428df6e8368d
Author: Alexey Serbin <alexey@apache.org>
AuthorDate: Fri Mar 1 13:18:05 2019 -0800

    [master] log on longer location mapping command runs
    
    This patch adds INFO logging when it takes more than 1 second
    to run the location mapping command.
    
    Change-Id: Icc420c704b1fad8bddd54285f1a9aa12b71095f0
    Reviewed-on: http://gerrit.cloudera.org:8080/12644
    Reviewed-by: Will Berkeley <wdberkeley@gmail.com>
    Tested-by: Kudu Jenkins
---
 src/kudu/master/location_cache.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/kudu/master/location_cache.cc b/src/kudu/master/location_cache.cc
index b79b259..f4f5378 100644
--- a/src/kudu/master/location_cache.cc
+++ b/src/kudu/master/location_cache.cc
@@ -32,6 +32,7 @@
 #include "kudu/gutil/strings/split.h"
 #include "kudu/gutil/strings/strip.h"
 #include "kudu/gutil/strings/substitute.h"
+#include "kudu/util/stopwatch.h"
 #include "kudu/util/subprocess.h"
 #include "kudu/util/trace.h"
 
@@ -127,6 +128,7 @@ Status LocationCache::GetLocationFromLocationMappingCmd(const string&
cmd,
   }
   argv.push_back(key);
   string stderr, location_temp;
+  SCOPED_LOG_SLOW_EXECUTION(WARNING, 1000, "running location mapping command");
   Status s = Subprocess::Call(argv, /*stdin_in=*/"", &location_temp, &stderr);
   if (!s.ok()) {
     return Status::RuntimeError(


Mime
View raw message