impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Casey Ching (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) Kudu: Update DCHECK about plan nodes that don't use runtime filters
Date Fri, 29 Apr 2016 01:04:38 GMT
Casey Ching has uploaded a new change for review.

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

Change subject: Kudu: Update DCHECK about plan nodes that don't use runtime filters
......................................................................

Kudu: Update DCHECK about plan nodes that don't use runtime filters

A DCHECK that expected that when runtime filters were set, the only plan
node that wouldn't use them is an HBase scan node. Kudu also doesn't use
runtime filters so this change updates the DCHECK.

The TODO was also updated. I'm guessing both Kudu and HBase should
eventually make use of runtime filters.

Change-Id: Ice6f7efaa701b60d0e5b75b8696d7a6e6ce96de6
---
M be/src/runtime/coordinator.cc
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/00/2900/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2900
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ice6f7efaa701b60d0e5b75b8696d7a6e6ce96de6
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <casey@cloudera.com>

Mime
View raw message