AmandeepSingh285 commented on PR #3695: URL: https://github.com/apache/celeborn/pull/3695#issuecomment-4554353445
Thanks @RexXiong for the review. 1. Recovery after failed reopen - The goal of the change was to have RocksDB automatically recovered rather than entering a terminal state. In the current state as well, in case of any failures when RocksDB enters a Read only mode, a manual intervention is required to have metadata updates unblocked which I tried to solve through this PR. 2. This is true that the test is relying on the version entry written. I can update the same with a small change if required. -- 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]
