rdblue commented on pull request #4071: URL: https://github.com/apache/iceberg/pull/4071#issuecomment-1036618601
@amogh-jahagirdar, I agree that we will probably want to turn the API into a transaction. I think that's not a big deal. Mainly, I'm wondering if you agree that we should combine the ManageSnapshots tasks with the UpdateSnapshotRefs tasks. They seem to me like things you'd want to combine, but I want to make sure you agree. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
