Mailing-List: contact reviews-help@impala.incubator.apache.org; run by ezmlm
Precedence: bulk
Message-Id: <201710301941.v9UJfGen024640@ip-10-146-233-104.ec2.internal>
Date: Mon, 30 Oct 2017 19:41:16 +0000
From: "Vuk Ercegovac (Code Review)" <gerrit@cloudera.org>
To: Michael Brown <mikeb@cloudera.com>, Philip Zeyliger <philip@cloudera.com>,
        Balazs Jeszenszky <jeszyb@gmail.com>,
        Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>,
        Alex Behm <alex.behm@cloudera.com>, Dan Hecht <dhecht@cloudera.com>,
        impala-cr@cloudera.com, reviews@impala.incubator.apache.org
Subject: =?UTF-8?Q?=5BImpala-ASF-CR=5D_IMPALA-4704=3A_Turns_on_client_connections_when_local_catalog_initialized=2E=0A?=
In-Reply-To: <gerrit.1507075635000.I52b881cba18a7e4533e21a78751c2e35c3d4c8a6@gerrit.cloudera.org>
References: <gerrit.1507075635000.I52b881cba18a7e4533e21a78751c2e35c3d4c8a6@gerrit.cloudera.org>
Reply-To: vercegovac@cloudera.com, impala-cr@cloudera.com, marcelk@gmail.com,
        dhecht@cloudera.com, philip@cloudera.com, alex.behm@cloudera.com,
        reviews@impala.incubator.apache.org, mikeb@cloudera.com,
        dtsirogiannis@cloudera.com, jeszyb@gmail.com
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
User-Agent: Gerrit/2.14.2
Content-Type: multipart/alternative; boundary="jRPTfq4n33Y="; charset=UTF-8
archived-at: Mon, 30 Oct 2017 19:41:24 -0000

--jRPTfq4n33Y=
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hello Michael Brown, Philip Zeyliger, Balazs Jeszenszky, Dimitris Tsirogian=
nis, Alex Behm, Dan Hecht, 

I'd like you to reexamine a change=2E Please v=
isit

    http://gerrit=2Ecloudera=2Eorg:8080/8202

to look at the new patc=
h set (#15)=2E

Change subject: IMPALA-4704: Turns on client connections wh=
en local catalog initialized=2E
=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=
=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=
=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=2E=
=2E=2E=2E=2E=2E=2E

IMPALA-4704: Turns on client connections when local cat=
alog initialized=2E

Currently, impalad starts beeswax and hs2 servers even=
 if the
catalog has not yet been initialized=2E As a result, client
connect=
ions see an error message stating that the impalad
is not yet ready=2E

Thi=
s patch changes the impalad startup sequence to wait
until the catalog is r=
eceived before opening beeswax and hs2 ports
and starting their servers=2E
=

Testing:
- python e2e tests that start a cluster without a catalog
  and c=
heck that client connections are rejected as expected=2E

Change-Id: I52b88=
1cba18a7e4533e21a78751c2e35c3d4c8a6
---
M be/src/benchmarks/expr-benchmark=
=2Ecc
M be/src/common/global-flags=2Ecc
M be/src/service/frontend=2Ecc
M be=
/src/service/frontend=2Eh
M be/src/service/impala-server=2Ecc
M be/src/serv=
ice/impala-server=2Eh
M be/src/service/impalad-main=2Ecc
M be/src/testutil/=
in-process-servers=2Ecc
M be/src/testutil/in-process-servers=2Eh
M bin/star=
t-impala-cluster=2Epy
M fe/src/main/java/org/apache/impala/service/Frontend=
=2Ejava
M fe/src/main/java/org/apache/impala/service/JniFrontend=2Ejava
M f=
e/src/test/java/org/apache/impala/service/FrontendTest=2Ejava
M tests/commo=
n/custom_cluster_test_suite=2Epy
M tests/common/impala_cluster=2Epy
M tests=
/common/impala_service=2Epy
A tests/custom_cluster/test_catalog_wait=2Epy
M=
 tests/custom_cluster/test_coordinators=2Epy
18 files changed, 306 insertio=
ns(+), 110 deletions(-)


  git pull ssh://gerrit=2Ecloudera=2Eorg:29418/Im=
pala-ASF refs/changes/02/8202/15
-- 
To view, visit http://gerrit=2Eclouder=
a=2Eorg:8080/8202
To unsubscribe, visit http://gerrit=2Ecloudera=2Eorg:8080=
/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageT=
ype: newpatchset
Gerrit-Change-Id: I52b881cba18a7e4533e21a78751c2e35c3d4c8a=
6
Gerrit-Change-Number: 8202
Gerrit-PatchSet: 15
Gerrit-Owner: Vuk Ercegova=
c <vercegovac@cloudera=2Ecom>
Gerrit-Reviewer: Alex Behm <alex=2Ebehm@cloud=
era=2Ecom>
Gerrit-Reviewer: Balazs Jeszenszky <jeszyb@gmail=2Ecom>
Gerrit-R=
eviewer: Dan Hecht <dhecht@cloudera=2Ecom>
Gerrit-Reviewer: Dimitris Tsirog=
iannis <dtsirogiannis@cloudera=2Ecom>
Gerrit-Reviewer: Michael Brown <mikeb=
@cloudera=2Ecom>
Gerrit-Reviewer: Philip Zeyliger <philip@cloudera=2Ecom>
G=
errit-Reviewer: Vuk Ercegovac <vercegovac@cloudera=2Ecom>

--jRPTfq4n33Y=--