pvary commented on PR #15151: URL: https://github.com/apache/iceberg/pull/15151#issuecomment-3826030562
> Thanks @Guosmilesmile! I appreciate your work on this. > > I think we can remove the recovery lock, if we switch to this design: I don't really like the solution: - It would need way more messages than a simple release lock solution - It would mean that we need to have constant communication with the Job Manager when we are waiting for the lock I don't think that the recovery lock is that complicated to make it worth to accept the messaging overhead and also add state. -- 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]
