priyeshkaratha commented on PR #11187: URL: https://github.com/apache/ozone/pull/11187#issuecomment-6261061665
Thanks @smengcl. I reworked this based on your last two comments. The summary is now reloaded in notifyLeaderReady instead of notifyLeaderChanged, and summary updates are rejected until the reload is done, so no update can be based on stale counters. This is coordinated with a flag under summaryLock, not a lock: notifyLeaderReady runs on the apply thread, so it must not wait on DeletedBlockLogImpl (that would deadlock with deleteKeyBlocks waiting on Ratis). inFlightOps and flushAndRun are removed. -- 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]
