apache / iceberg-rust

Apache Iceberg
https://rust.iceberg.apache.org/
Apache License 2.0
477 stars 97 forks source link

add `InclusiveProjection` Visitor #321

Closed sdd closed 2 months ago

sdd commented 3 months ago

This PR has been broken out of https://github.com/apache/iceberg-rust/pull/241 as it was getting too large.

The InclusiveProjection is used in the process of filtering manifest files in table scans. It projects the BoundPredicate that is provided as the filter in a table scan into a Predicate that can be used for filtering the partition specs of manifests.

This PR depends on https://github.com/apache/iceberg-rust/pull/320.

sdd commented 3 months ago

Updated to align with latest BoundPredicateVisitor iteration in https://github.com/apache/iceberg-rust/pull/320

sdd commented 2 months ago

Closing this one in favour of the alternate PR based on the BoundPredicateVisitor design that got merged (https://github.com/apache/iceberg-rust/pull/335)