drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mehant Baid" <baid.meh...@gmail.com>
Subject Re: Review Request 36630: DRILL-3503: Make PruneScanRule pluggable
Date Fri, 31 Jul 2015 03:35:27 GMT

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

(Updated July 31, 2015, 3:35 a.m.)


Review request for drill and Aman Sinha.


Changes
-------

Rebased on latest master. Moved a change from the patch for 3121 here.


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


Repository: drill-git


Description
-------

Added an interface to abstract away partitioning scheme away from the partition pruning rule.
Removed some of the redundant logic in PruneScanRule.


Diffs (updated)
-----

  contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionDescriptor.java
8307dff 
  exec/java-exec/src/main/java/org/apache/drill/exec/planner/DFSPartitionLocation.java PRE-CREATION

  exec/java-exec/src/main/java/org/apache/drill/exec/planner/FileSystemPartitionDescriptor.java
9ad14b1 
  exec/java-exec/src/main/java/org/apache/drill/exec/planner/ParquetPartitionDescriptor.java
127e70a 
  exec/java-exec/src/main/java/org/apache/drill/exec/planner/PartitionDescriptor.java 35fdae9

  exec/java-exec/src/main/java/org/apache/drill/exec/planner/PartitionLocation.java PRE-CREATION

  exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillPushPartitionFilterIntoScan.java
b83cedd 
  exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRuleSets.java daa7276

  exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/partition/ParquetPruneScanRule.java
PRE-CREATION 
  exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/partition/PruneScanRule.java
5b5e4bc 

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


Testing
-------


Thanks,

Mehant Baid


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