sarvekshayr commented on PR #7682: URL: https://github.com/apache/ozone/pull/7682#issuecomment-2586598376
In my opinion, it would be better to make the `--dry-run` option the default behaviour. This allows users to see what changes would be performed before executing the actual repair, reducing the risk of accidental modifications to the database. While users can explicitly specify the `--dry-run` option to preview changes, making it the default ensures it won't be overlooked. Since the primary focus of this PR is to add the `--dry-run` option for all repair commands (and any commands to be added in the future), we can consider making `--dry-run` the default behaviour in a follow-up 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org