ivandika3 commented on PR #10525: URL: https://github.com/apache/ozone/pull/10525#issuecomment-4767366216
@ashishkumar50 Note that this is technically a no-op patch since the SCM transition container lifecycle uses `markStateTransitionIdempotent` to prevent this invalid transition. However, I think the default behavior of crashing SCM is quite harsh. I was working on our internal feature that introduces a separate lifecycle and encountered the SCM crash issue which was unexpected. -- 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]
