search

apache / iceberg-rust

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

Add a few getters that are required for the evaluating manifests in TableScan #318

Closed sdd closed 3 months ago

sdd commented 3 months ago

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

It should be pretty uncontroversial, adding some getters that are required in the manifest evaluation process.

Additionally the chrono dep in Cargo.toml was changed from being pinned to 0.4.34 to being ~0.4.34 to permit compatible updates as they arise.

sdd commented 3 months ago

@liurenjie1024 and @marvinlanhenke - small PR split out from https://github.com/apache/iceberg-rust/pull/241 ready for review :-)

Actually ignore this, I'll factor these out into the PRs where they actually get used