apache / iceberg-rust

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

refactor: cache partition_schema in `fn plan_files()` #362

Closed marvinlanhenke closed 2 months ago

marvinlanhenke commented 2 months ago

Which issue does this PR close?

Closes #361

Rationale for this change

This is a left-over part from #360 Tries to solve the unresolved issue of creating a new PartitionSchema in every iteration / or for each ManifestFile.

What changes are included in this PR?

Are these changes tested?

Yes. Unit tests from before.

marvinlanhenke commented 2 months ago

@sdd @liurenjie1024 PTAL