You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by da...@apache.org on 2017/11/07 19:15:17 UTC
[5/6] kudu git commit: mini_hms: bump start timeout to 60sec
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 <da...@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 <to...@apache.org>
Authored: Mon Nov 6 15:52:19 2017 -0800
Committer: Todd Lipcon <to...@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 {