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 C1444200BD5 for ; Thu, 8 Dec 2016 18:59:02 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id BFDD1160B1F; Thu, 8 Dec 2016 17:59:02 +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 14B98160B0A for ; Thu, 8 Dec 2016 18:59:01 +0100 (CET) Received: (qmail 78327 invoked by uid 500); 8 Dec 2016 17:59:00 -0000 Mailing-List: contact reviews-help@impala.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list reviews@impala.incubator.apache.org Received: (qmail 78309 invoked by uid 99); 8 Dec 2016 17:59:00 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd4-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 Dec 2016 17:59:00 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd4-us-west.apache.org (ASF Mail Server at spamd4-us-west.apache.org) with ESMTP id 21A9FC036F for ; Thu, 8 Dec 2016 17:59:00 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd4-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.363 X-Spam-Level: X-Spam-Status: No, score=0.363 tagged_above=-999 required=6.31 tests=[RDNS_DYNAMIC=0.363, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd4-us-west.apache.org [10.40.0.11]) (amavisd-new, port 10024) with ESMTP id GalA2GRS44G8 for ; Thu, 8 Dec 2016 17:58:59 +0000 (UTC) Received: from ip-10-146-233-104.ec2.internal (ec2-75-101-130-251.compute-1.amazonaws.com [75.101.130.251]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTPS id 0AD755F5C6 for ; Thu, 8 Dec 2016 17:58:58 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by ip-10-146-233-104.ec2.internal (8.14.4/8.14.4) with ESMTP id uB8Hwvl0006963; Thu, 8 Dec 2016 17:58:57 GMT Message-Id: <201612081758.uB8Hwvl0006963@ip-10-146-233-104.ec2.internal> Date: Thu, 8 Dec 2016 17:58:56 +0000 From: "Jim Apple (Code Review)" To: Jim Apple , Tim Armstrong , impala-cr@cloudera.com, reviews@impala.incubator.apache.org CC: Alex Behm Reply-To: jbapple-impala@apache.org X-Gerrit-MessageType: newpatchset Subject: =?UTF-8?Q?=5BImpala-ASF-CR=5D_IMPALA-4588=3A_Enable_starting_the_minicluster_when_offline=0A?= X-Gerrit-Change-Id: Ifbb5babebb0ca6d2553be1b001e20e2270e052b6 X-Gerrit-ChangeURL: X-Gerrit-Commit: b38a08ccbb06c80c7567dd2f8cebf77f77d3ab34 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/2.12.2 archived-at: Thu, 08 Dec 2016 17:59:02 -0000 Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5412 to look at the new patch set (#3). Change subject: IMPALA-4588: Enable starting the minicluster when offline ...................................................................... IMPALA-4588: Enable starting the minicluster when offline IMPALA-4553 made ntp-wait succeed before kudu would start, assuming ntp-wait was installed, in order to prevent a litany of errors on ec2 about unsynchronized clocks. This patch disables that waiting if no internet connection is detected in order to make it possible to start the minicluster when offline. Change-Id: Ifbb5babebb0ca6d2553be1b001e20e2270e052b6 --- M testdata/cluster/admin 1 file changed, 13 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/12/5412/3 -- To view, visit http://gerrit.cloudera.org:8080/5412 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifbb5babebb0ca6d2553be1b001e20e2270e052b6 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Jim Apple Gerrit-Reviewer: Tim Armstrong