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

Reply via email to