impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henry Robinson (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3245 / IMPALA-3305: Fix crash with global filters when NUM_NODES=1
Date Wed, 27 Apr 2016 20:43:56 GMT
Hello Marcel Kornacker, Internal Jenkins, Alex Behm,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/2721

to look at the new patch set (#6).

Change subject: IMPALA-3245 / IMPALA-3305: Fix crash with global filters when NUM_NODES=1
......................................................................

IMPALA-3245 / IMPALA-3305: Fix crash with global filters when NUM_NODES=1

Filter locality was not correctly set when NUM_NODES=1 (and therefore no
distributed plan was created). The default locality should be 'local'.

This patch also fixes a bug where the initial filter routing table
wasn't printed when NUM_NODES=1.

Change-Id: I7b9a6bcc64ca6ec5fd51d63815cea25de866ef93
---
M be/src/runtime/coordinator.cc
M be/src/runtime/coordinator.h
M fe/src/main/java/com/cloudera/impala/planner/DistributedPlanner.java
M fe/src/main/java/com/cloudera/impala/planner/RuntimeFilterGenerator.java
M testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test
5 files changed, 153 insertions(+), 117 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/21/2721/6
-- 
To view, visit http://gerrit.cloudera.org:8080/2721
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7b9a6bcc64ca6ec5fd51d63815cea25de866ef93
Gerrit-PatchSet: 6
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Marcel Kornacker <marcel@cloudera.com>

Mime
View raw message