Mailing-List: contact reviews-help@impala.incubator.apache.org; run by ezmlm
Precedence: bulk
Message-Id: <201710202256.v9KMuv7a013698@ip-10-146-233-104.ec2.internal>
Date: Fri, 20 Oct 2017 22:56:57 +0000
From: "Vuk Ercegovac (Code Review)" <gerrit@cloudera.org>
To: Philip Zeyliger <philip@cloudera.com>,
        Balazs Jeszenszky <jeszyb@gmail.com>,
        Dimitris Tsirogiannis <dtsirogiannis@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,
        reviews@impala.incubator.apache.org, 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="Qd7HEGakrNM="; charset=UTF-8
archived-at: Fri, 20 Oct 2017 22:57:03 -0000

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

Hello Philip Zeyliger, Balazs Jeszenszky, Dimitris Tsirogiannis, Dan Hecht,=
 

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

    http://gerrit=2E=
cloudera=2Eorg:8080/8202

to look at the new patch set (#10)=2E

Change sub=
ject: IMPALA-4704: Turns on client connections when local catalog initializ=
ed=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 catalog initialized=2E

Curren=
tly, impalad starts beeswax and hs2 servers even if the
catalog has not yet=
 been initialized=2E As a result, client
connections see an error message s=
tating that the impalad
is not yet ready=2E

This patch changes the impalad=
 startup sequence to wait
until the catalog is received before opening bees=
wax and hs2 ports
and starting their servers=2E

Testing:
- python e2e test=
s that start a cluster without a catalog
  and check that client connection=
s are rejected as expected=2E

Change-Id: I52b881cba18a7e4533e21a78751c2e35=
c3d4c8a6
---
M be/src/service/frontend=2Ecc
M be/src/service/frontend=2Eh
M=
 be/src/service/impala-server=2Ecc
M be/src/testutil/in-process-servers=2Ec=
c
M bin/start-impala-cluster=2Epy
M fe/src/main/java/org/apache/impala/serv=
ice/Frontend=2Ejava
M fe/src/main/java/org/apache/impala/service/JniFronten=
d=2Ejava
M fe/src/test/java/org/apache/impala/service/FrontendTest=2Ejava
M=
 tests/common/custom_cluster_test_suite=2Epy
M tests/common/impala_cluster=
=2Epy
M tests/common/impala_service=2Epy
A tests/custom_cluster/test_catalo=
g_wait=2Epy
M tests/custom_cluster/test_coordinators=2Epy
13 files changed,=
 169 insertions(+), 71 deletions(-)


  git pull ssh://gerrit=2Ecloudera=2E=
org:29418/Impala-ASF refs/changes/02/8202/10
-- 
To view, visit http://gerr=
it=2Ecloudera=2Eorg:8080/8202
To unsubscribe, visit http://gerrit=2Eclouder=
a=2Eorg:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Ger=
rit-MessageType: newpatchset
Gerrit-Change-Id: I52b881cba18a7e4533e21a78751=
c2e35c3d4c8a6
Gerrit-Change-Number: 8202
Gerrit-PatchSet: 10
Gerrit-Owner: =
Vuk Ercegovac <vercegovac@cloudera=2Ecom>
Gerrit-Reviewer: Balazs Jeszenszk=
y <jeszyb@gmail=2Ecom>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera=2Ecom>
G=
errit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera=2Ecom>
Gerrit=
-Reviewer: Philip Zeyliger <philip@cloudera=2Ecom>
Gerrit-Reviewer: Vuk Erc=
egovac <vercegovac@cloudera=2Ecom>

--Qd7HEGakrNM=--