xichen01 commented on PR #4574: URL: https://github.com/apache/ozone/pull/4574#issuecomment-1511057905
> > > @xichen01 , do you aware the common value of applyTransaction and writeStateMachineData? Are their value always higher than 1ms? > > > > > > I mainly want to standardize the units of metrics, in `CMSMetrics`, some metrics are in `ms` and some are in `ns`, which is not directly known from the names of the metrics, which may cause misunderstanding. > > According to my observation the average value `writeStateMachineData` is usually 1~10ms, the average value of `applyTransaction` is usually less than 1ms, but the maximum value of both is more than 1ms. > > For values under 1ms, whether we can consider as normal, need not to care? So a uniform unit of metrics is more friendly. > > If "applyTransaction" is usually less than 1ms, then it will lose accuracy from ns to ms. If the goal is unify the time unit, then maybe switch other metrics from ms to `ns` is what we can go. It's a good idea that switch other metrics from ms to `ns`. But is there a more direct way to confirm the units of a metric? Currently, if I want to confirm the units of an metric, I can only check the code to confirm -- 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]
