impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Internal Jenkins (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3521: Impalad should communicate with the statestore after binding to the hs2 and besswax ports
Date Wed, 20 Jul 2016 03:00:32 GMT
Internal Jenkins has submitted this change and it was merged.

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
Reviewed-on: http://gerrit.cloudera.org:8080/3264
Reviewed-by: anujphadke <aphadke@cloudera.com>
Tested-by: Internal Jenkins
---
M be/src/service/impalad-main.cc
1 file changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Internal Jenkins: Verified
  anujphadke: Looks good to me, approved



-- 
To view, visit http://gerrit.cloudera.org:8080/3264
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5c09dafa5475978424e63bc936c0cdf4940df8d7
Gerrit-PatchSet: 5
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: anujphadke <aphadke@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: anujphadke <aphadke@cloudera.com>

Mime
View raw message