apache / bookkeeper / 21 / 1

72%

master: 72%

push

jenkins

<a href="https://github.com/apache/bookkeeper/commit/<a class=hub.com/apache/bookkeeper/commit/4578e6d65a304c870fcba013099badd6018c95de">4578e6d65<a href="https://github.com/apache/bookkeeper/commit/4578e6d65a304c870fcba013099badd6018c95de">&quot;&gt;Fixed Journal stats names

In `BOOKKEEPER-1009: Use multiple journals in bookie` </a><a class="double-link" href="https://github.com/apache/bookkeeper/commit/<a class="double-link" href="https://github.com/apache/bookkeeper/commit/123eccd435a4a96a9147ed4a24efbe9025fe79ba">123eccd43</a>">123eccd43</a><a href="https://github.com/apache/bookkeeper/commit/4578e6d65a304c870fcba013099badd6018c95de"> there was a change in the metrics name that would be affecting also user not running with multiple journal.

It is a bit inconvenient to aggregate the stats in the metrics collector (how to aggregate 99pct latency for example).

I think the best option is to have a single metric even when using multiple journal threads.

Author: Matteo Merli &lt;mmerli@apache.org&gt;

Reviewers: Ivan Kelly &lt;ivank@apache.org&gt;, Enrico Olivelli &lt;eolivelli@gmail.com&gt;, Sijie Guo <sijie@apache.org>

This closes #1250 from merlimat/journal-metric-names

22872 of 31374 relevant lines covered (72.9%)

0.73 hits per line