errose28 commented on PR #7450: URL: https://github.com/apache/ozone/pull/7450#issuecomment-2483461798
Thanks for adding this Attila. +1 for this `ozone debug version` also including the overall ozone version, making it like a lower level extension of `ozone version`. I like this simple implementation where the command just prints the component version of the current `ozone` executable that is run since it is a `debug` subcommand and those usually run totally local. I think we just need to make the help message a little clearer in explaining this. For example, one might expect one run of `ozone debug version` from the client machine to identify a mixed client/server version scenario, but this is not the case. It might be good to add a startup log message for component versions like we have for layout versions, but we can do that in a separate change if we want. -- 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]
