----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53687/#review155767 -----------------------------------------------------------
Ship it! Master (8a9939f) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Nov. 12, 2016, 1:49 a.m., Reza Motamedi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53687/ > ----------------------------------------------------------- > > (Updated Nov. 12, 2016, 1:49 a.m.) > > > Review request for Aurora and Joshua Cohen. > > > Repository: aurora > > > Description > ------- > > track percentile values for timed sliding stats. AURORA-118 > > > Diffs > ----- > > commons/src/main/java/org/apache/aurora/common/inject/TimedInterceptor.java > a4c4240b67cafb108ca587ad86d9fb9d9bb1553d > commons/src/main/java/org/apache/aurora/common/stats/SlidingStats.java > f7a5ae41e307627fc55157758e9b7cdd861c3268 > commons/src/main/java/org/apache/aurora/common/stats/SlidingTimedStats.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/53687/diff/ > > > Testing > ------- > > This is a patch for the request in > https://issues.apache.org/jira/browse/AURORA-118. > > SlidingTimedStats extends SlidingStats and contains aPercentile for tracking > a finer view of the timing stats. Percentile was already implemented to track > RequestStats and I reused it to track any timing stats captured via @Timed > annotation. > > > Thanks, > > Reza Motamedi > >
