Wenzhe Zhou has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/17502 )

Change subject: IMPALA-10197 (Part 2): Add KUDU_REPLICA_SELECTION query option
......................................................................

IMPALA-10197 (Part 2): Add KUDU_REPLICA_SELECTION query option

The previous patch added a new test case in PlannerTest for Kudu,
which check for specific number of scan nodes and instances for
distributed plan with query option KUDU_REPLICA_SELECTION set as
LEADER_ONLY.
However, the leadership isn't deterministic since each Kudu partition
has its own Raft group and the leaders of these groups are not exactly
balanced across a cluster. There's no guarantee that we'll get a
certain number of leader nodes for a query which access multiple KUDU
partitions. This make the unit-test flaky.

This patch removed the queries which access multiple KUDU partitions
for the test case with KUDU_REPLICA_SELECTION set as LEADER_ONLY. Also
removed the corresponding queries for the test case with
KUDU_REPLICA_SELECTION set as CLOSEST_REPLICA.

Testing:
 - Reran the Planner test and verified the issue did not happen.

Change-Id: I3e23667c06c273a261e03de3d81fc7ee1f6b0682
---
M 
testdata/workloads/functional-planner/queries/PlannerTest/kudu-replica-selection-closest-replica.test
M 
testdata/workloads/functional-planner/queries/PlannerTest/kudu-replica-selection-leader-only.test
2 files changed, 0 insertions(+), 215 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3e23667c06c273a261e03de3d81fc7ee1f6b0682
Gerrit-Change-Number: 17502
Gerrit-PatchSet: 2
Gerrit-Owner: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>

Reply via email to