drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aman Sinha" <asi...@maprtech.com>
Subject Review Request 32523: DRILL-2568: Conditionally drop filter during partition pruning
Date Thu, 26 Mar 2015 03:49:23 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32523/
-----------------------------------------------------------

Review request for drill and Jacques Nadeau.


Bugs: DRILL-2568
    https://issues.apache.org/jira/browse/DRILL-2568


Repository: drill-git


Description
-------

Drop the filter plan node if all conjuncts in the filter have been pushed as part of partition
pruning, except for the situation where the new set of files is empty - we add a single file
in that case, so the Filter is preserved in that case.


Diffs
-----

  exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/partition/PruneScanRule.java
b8c9ebf 
  exec/java-exec/src/test/java/org/apache/drill/exec/expr/TestPrune.java d15555e 

Diff: https://reviews.apache.org/r/32523/diff/


Testing
-------

Manual inspection of Explain plans for several types of queries.  Running regression tests..


Thanks,

Aman Sinha


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message