Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 5244E200D5B for ; Tue, 7 Nov 2017 20:15:16 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id 509E2160BEB; Tue, 7 Nov 2017 19:15:16 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 9E702160C01 for ; Tue, 7 Nov 2017 20:15:15 +0100 (CET) Received: (qmail 6610 invoked by uid 500); 7 Nov 2017 19:15:14 -0000 Mailing-List: contact commits-help@kudu.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@kudu.apache.org Delivered-To: mailing list commits@kudu.apache.org Received: (qmail 6540 invoked by uid 99); 7 Nov 2017 19:15:14 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 07 Nov 2017 19:15:14 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id D8E43DFE61; Tue, 7 Nov 2017 19:15:13 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: danburkert@apache.org To: commits@kudu.apache.org Date: Tue, 07 Nov 2017 19:15:17 -0000 Message-Id: In-Reply-To: <9b3255502c6b470c9051e75f82c9df20@git.apache.org> References: <9b3255502c6b470c9051e75f82c9df20@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [5/6] kudu git commit: mini_hms: bump start timeout to 60sec archived-at: Tue, 07 Nov 2017 19:15:16 -0000 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 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 Authored: Mon Nov 6 15:52:19 2017 -0800 Committer: Todd Lipcon 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 {