impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dimitris Tsirogiannis (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4041: Limit catalog and admission control updates to coordinators
Date Wed, 22 Mar 2017 18:33:39 GMT
Dimitris Tsirogiannis has uploaded a new patch set (#4).

Change subject: IMPALA-4041: Limit catalog and admission control updates to coordinators

IMPALA-4041: Limit catalog and admission control updates to coordinators

With this commit we add the ability to limit catalog updates to a
limited set of coordinator nodes. A new startup option, termed
'is_coordinator' is added to indicate if a node is a coordinator.
Coordinators accept connections through HS2 and Beeswax interfaces
and can also participate in query execution. Non-coordinator nodes
do not receive catalog updates from the statestore, do not initialize
a query scheduler and cannot accept Beeswax and HS2 client connections.

- Added a custom cluster test that launches a cluster in which the
number of coordinators is less than the cluster size and runs a number
of smoke queries.
- Successfully run exhaustive tests.

Change-Id: I5f2c74abdbcd60ac050efa323616bd41182ceff3
M be/src/runtime/
M be/src/runtime/exec-env.h
M be/src/scheduling/
M be/src/scheduling/admission-controller.h
M be/src/scheduling/
M be/src/scheduling/
M be/src/scheduling/scheduler.h
M be/src/service/
M be/src/service/impala-server.h
M be/src/service/
M be/src/service/
M be/src/testutil/in-process-servers.h
M bin/
M tests/common/
M tests/common/
A tests/custom_cluster/
16 files changed, 365 insertions(+), 198 deletions(-)

  git pull ssh:// refs/changes/44/6344/4
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5f2c74abdbcd60ac050efa323616bd41182ceff3
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Henry Robinson <>
Gerrit-Reviewer: Marcel Kornacker <>
Gerrit-Reviewer: Matthew Jacobs <>

View raw message