c-thiel commented on code in PR #645: URL: https://github.com/apache/iceberg-rust/pull/645#discussion_r1825848112
########## crates/iceberg/src/spec/partition.rs: ########## @@ -131,50 +205,22 @@ impl PartitionSpec { /// * Field names /// * Source column ids /// * Transforms - pub fn is_compatible_with(&self, other: &UnboundPartitionSpec) -> bool { + pub fn is_compatible_with_unbound(&self, other: &UnboundPartitionSpec) -> bool { Review Comment: I moved the implementation to `BoundPartitionSpec` but comparing still to `SchemalessPartitionSpec` and not `BoundPartitionSpec`. I need to use this method to check if a given spec already exists. The spec is passed as unbound but then bound against the `current_schema`. All old specs are however `SchemalessPartitionSpec ` and I would really like to avoid re-binding them all, as this is costly - we might have arbitrarily many schema specs as they are never cleaned u. Also the additional bind-check gives us no additional information. If the `BoundPartitionSpec` was able to bind against the schema, so will the `SchemalessPartitionSpec`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org