shanielh commented on PR #10736: URL: https://github.com/apache/iceberg/pull/10736#issuecomment-2253744447
> Ah just saw/remembered [#10736 (comment)](https://github.com/apache/iceberg/pull/10736#issuecomment-2244231366), the write APIs Iceberg exposes allow passing in a given spec ID. So you're not really looking to set the spec as current but rather just leverage the existence of the spec in the write APIs. That's reasonable. > > Cool, I think I'd still advocate for a different method name. [#10736 (comment)](https://github.com/apache/iceberg/pull/10736#discussion_r1693722945) > > After that, I think the PR looks good to me. Ok, I've modified the method name and squashed all commits into one. Thanks! -- 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
