swamirishi commented on PR #8157: URL: https://github.com/apache/ozone/pull/8157#issuecomment-2798227577
> @peterxcli Had a discussion with @hemantk-12 we should go ahead with enforcing hard limits on the number of snapshots. For that the number of snapshot creates inflight would have to be considered. This counter should be only relevant on leader. So on every leader switch this counter needs to be reset, which can be detected by implementing As a side note we can actually implement a notifier similar to what SCM has to notify status of OM has changed, maybe resuing the same interface is also a good idea. What do you think @hemantk-12 ? https://github.com/apache/ozone/blob/0e109d19eaaf82f8c5220d506c4b3d3a2ab03899/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/ha/SCMService.java#L27-L31 -- 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]
