kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danburk...@apache.org
Subject [5/6] kudu git commit: mini_hms: bump start timeout to 60sec
Date Tue, 07 Nov 2017 19:15:17 GMT
mini_hms: bump start timeout to 60sec

Change-Id: Icd791377207563dc91b36d7f21fcfedfb722fb7b
Reviewed-on: http://gerrit.cloudera.org:8080/8485
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <danburkert@apache.org>


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

Branch: refs/heads/master
Commit: 0e3b0cecfe40e62f4a5d75e7e6e0bae026b34275
Parents: fde2b56
Author: Todd Lipcon <todd@apache.org>
Authored: Mon Nov 6 15:52:19 2017 -0800
Committer: Todd Lipcon <todd@apache.org>
Committed: Tue Nov 7 17:44:34 2017 +0000

----------------------------------------------------------------------
 src/kudu/hms/mini_hms.cc | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/0e3b0cec/src/kudu/hms/mini_hms.cc
----------------------------------------------------------------------
diff --git a/src/kudu/hms/mini_hms.cc b/src/kudu/hms/mini_hms.cc
index 392033b..75ef0b7 100644
--- a/src/kudu/hms/mini_hms.cc
+++ b/src/kudu/hms/mini_hms.cc
@@ -43,6 +43,8 @@ using std::string;
 using std::unique_ptr;
 using strings::Substitute;
 
+static constexpr int kHmsStartTimeoutMs = 60000;
+
 namespace kudu {
 namespace hms {
 
@@ -73,7 +75,7 @@ Status FindHomeDir(const char* name, const string& bin_dir, string*
home_dir) {
 } // anonymous namespace
 
 Status MiniHms::Start() {
-  SCOPED_LOG_SLOW_EXECUTION(WARNING, 20000, "Starting HMS");
+  SCOPED_LOG_SLOW_EXECUTION(WARNING, kHmsStartTimeoutMs / 2, "Starting HMS");
   CHECK(!hms_process_);
 
   VLOG(1) << "Starting HMS";
@@ -118,7 +120,8 @@ Status MiniHms::Start() {
 
   // Wait for HMS to start listening on its ports and commencing operation.
   VLOG(1) << "Waiting for HMS ports";
-  return WaitForTcpBind(hms_process_->pid(), &port_, MonoDelta::FromSeconds(20));
+  return WaitForTcpBind(hms_process_->pid(), &port_,
+                        MonoDelta::FromMilliseconds(kHmsStartTimeoutMs));
 }
 
 Status MiniHms::CreateHiveSite(const string& tmp_dir) const {


Mime
View raw message