You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "anujphadke (Code Review)" <ge...@cloudera.org> on 2016/06/01 20:25:53 UTC

[Impala-CR](cdh5-trunk) IMPALA-3521: Impalad should communicate with the statestore after binding to the hs2 and besswax ports

anujphadke has uploaded a new patch set (#3).

Change subject: IMPALA-3521: Impalad should communicate with the statestore  after binding to the hs2 and besswax ports
......................................................................

IMPALA-3521: Impalad should communicate with the statestore  after
binding to the hs2 and besswax ports

Impalad on startup tries to connect with the statestore first and then
tries to open the beeswax and the hs2 ports. If the ports are already
bound to an another process, impalad will abort and statestore would
still mark this impalad as up. This patch changes the sequence in which
we perform these operations so that statestore is started after opening
the network ports. Any failures while starting the hs2 and beeswax
services should not result in false statestore membership updates.

Change-Id: I5c09dafa5475978424e63bc936c0cdf4940df8d7
---
M be/src/service/impalad-main.cc
1 file changed, 3 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/64/3264/3
-- 
To view, visit http://gerrit.cloudera.org:8080/3264
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5c09dafa5475978424e63bc936c0cdf4940df8d7
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: anujphadke <ap...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: anujphadke <ap...@cloudera.com>