smengcl commented on PR #10167: URL: https://github.com/apache/ozone/pull/10167#issuecomment-4374373452
> Regardless of how dangerous it is, debug commands are supposed to be read only so if we add this it needs to be under `ozone repair`. > > Regarding the use case, I'm not sure this is common enough to warrant a dangerous command like this being generally added to upstream. > > > This is no more dangerous than admin deleting the ozone metadata by accident TBH. > > It's generally understood that doing manual filesystem operations on Ozone's internal state is dangerous and may result in corruption. It is not as well understood that commands added and endorsed by maintainers can do the same. Fair. I will close the PR and it will live as a patch file in the JIRA. -- 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]
