Pooja Nilangekar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/10873


Change subject: IMPALA-6031: Distributed plan describes coordinator-only nodes 
as scanning
......................................................................

IMPALA-6031: Distributed plan describes coordinator-only nodes as scanning

The MembershipCallback function of ImpalaServer class was modified
to only send details of executor nodes to the frontend. Thus, while
generating scan plans, the frontend would not assign fragments to
coordinator-only nodes. The MembershipSnapshot class was renamed to
ExecutorMembershipSnapshot for better semantics.

Testing:
Added a new custom_cluster test where one impalad is a
coordinator-only node. The test verifies that the scan fragments
are assigned only to the executors.

Change-Id: I44af6b40099a495e13a0a5dc72c491d486d23aa8
---
M be/src/service/frontend.cc
M be/src/service/frontend.h
M be/src/service/impala-server.cc
M common/thrift/Frontend.thrift
M fe/src/main/java/org/apache/impala/planner/HBaseScanNode.java
M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M fe/src/main/java/org/apache/impala/service/JniFrontend.java
R fe/src/main/java/org/apache/impala/util/ExecutorMembershipSnapshot.java
M fe/src/test/java/org/apache/impala/planner/PlannerTestBase.java
M tests/custom_cluster/test_coordinators.py
11 files changed, 82 insertions(+), 44 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/73/10873/2
--
To view, visit http://gerrit.cloudera.org:8080/10873
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I44af6b40099a495e13a0a5dc72c491d486d23aa8
Gerrit-Change-Number: 10873
Gerrit-PatchSet: 2
Gerrit-Owner: Pooja Nilangekar <[email protected]>

Reply via email to