Skip to content

Commit 9382042

Browse files
vjkoskelaBrandonArp
authored andcommitted
Update PipelineConfiguration.java (#61)
Looks like mean is added twice to the set; I bet we meant median for the other entry. Should we set it to median or drop it entirely? The latter is completely backwards compatible.
1 parent 9644ca4 commit 9382042

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/main/java/com/arpnetworking/metrics/mad/configuration/PipelineConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@ public Builder setStatistics(final Map<String, Set<Statistic>> value) {
265265
@NotNull
266266
@NotEmpty
267267
private Set<Statistic> _timerStatistics = Sets.<Statistic>newHashSet(
268-
STATISTIC_FACTORY.getStatistic("mean"),
268+
STATISTIC_FACTORY.getStatistic("median"),
269269
STATISTIC_FACTORY.getStatistic("tp90"),
270270
STATISTIC_FACTORY.getStatistic("tp99"),
271271
STATISTIC_FACTORY.getStatistic("mean"),

0 commit comments

Comments
 (0)