advancedxy commented on PR #12161: URL: https://github.com/apache/iceberg/pull/12161#issuecomment-2635591315
@Fokko Thanks for bring this up. I'm supporting this to only allow multi-arg transforms for V3 onwards. When multi-arg transform was first added to the spec, V3 was far from completion and we want to make it possible for V2 to support that. I think it's good timing to reconsider that as the multi-arg transform is still lacking in the java side. > Other implementations might not be aware of this implementation and drop the 2nd argument onward: To clarify, this might not happen. All the existing transform should be the singe-arg transform. For multi-arg transforms, it shall be added to the spec first and other implementation should be aware of the new transforms if it's spec compliant. -- 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
