datafusion icon indicating copy to clipboard operation
datafusion copied to clipboard

Use parquet predicates for indexed fields

Open nl5887 opened this issue 3 years ago • 2 comments

Currently indexed fields are not supported, causing predicate_evaluation_errors and silently ignoring the predicate. This fix will add support for indexed fields.

nl5887 avatar Jun 12 '22 20:06 nl5887

Codecov Report

Merging #2724 (77beccd) into master (ed0fe8c) will increase coverage by 0.00%. The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           master    #2724   +/-   ##
=======================================
  Coverage   84.70%   84.70%           
=======================================
  Files         270      270           
  Lines       47262    47266    +4     
=======================================
+ Hits        40035    40039    +4     
  Misses       7227     7227           
Impacted Files Coverage Δ
datafusion/core/src/physical_optimizer/pruning.rs 93.48% <50.00%> (-0.29%) :arrow_down:
datafusion/expr/src/logical_plan/plan.rs 73.91% <0.00%> (+0.19%) :arrow_up:
datafusion/optimizer/src/utils.rs 32.79% <0.00%> (+0.53%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ed0fe8c...77beccd. Read the comment docs.

codecov-commenter avatar Jun 12 '22 20:06 codecov-commenter

Marking as draft to make it clear it is waiting for some tests -- please mark it ready for review when it is ready for review again. Thanks @nl5887 !

alamb avatar Jul 02 '22 11:07 alamb

This PR is more than 6 month old, so closing it down for now to clean up the PR list. Please reopen if this is a mistake and you plan to work on it more

alamb avatar Jan 14 '23 11:01 alamb