steveloughran commented on pull request #2323: URL: https://github.com/apache/hadoop/pull/2323#issuecomment-701491715
> error. This is solved when I add ".mean" at the end of the DurationTracker stat name. The same with Max(".max" has to be added) and Min(".min" has to be added) to fetch their values. > Was thinking if we could add it to the getter rather than having to append with stat name. I'm a bit reluctant to, as then people would ask about failures next. What could be handy would be something in the support class to get all stats for a duration (or null), with some struct to contain them all. e.g `fetchDurationStatistics(IOStatistics, key) -> {count, min, max ,mean, failed, failed.min, failed.max, failed.mean)`, and you'd then work off that. Seem good? Troublespot: what if only some of the values were found? they'd be null in the result. Maybe we'd let you ask for the success/failure stats separately, deal with it that way, and have some `boolean isComplete()` probe to check all are set ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org