Skip to content

Commit e6e61b0

Browse files
Merge branch '1.9.x' into 1.10.x
2 parents b782d08 + 201f054 commit e6e61b0

File tree

5 files changed

+7
-6
lines changed

5 files changed

+7
-6
lines changed

implementations/micrometer-registry-prometheus/src/main/java/io/micrometer/prometheus/PrometheusTimer.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ protected void recordNonNegative(long amount, TimeUnit unit) {
9494
max.record(nanoAmount, TimeUnit.NANOSECONDS);
9595

9696
if (histogram != null) {
97-
histogram.recordLong(TimeUnit.NANOSECONDS.convert(amount, unit));
97+
histogram.recordLong(nanoAmount);
9898
}
9999
}
100100

micrometer-core/src/main/java/io/micrometer/core/instrument/binder/logging/LogbackMetrics.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ public FilterReply decide(Marker marker, Logger logger, Level level, String form
206206
// and can return immediately and
207207
// avoid the relatively expensive ThreadLocal access below. See also logbacks
208208
// Logger.callTurboFilters().
209-
// Calling logger.isEnabledFor(level) might be sub-optimal since it cals this
209+
// Calling logger.isEnabledFor(level) might be sub-optimal since it calls this
210210
// filter again. This behavior caused a StackOverflowError in the past.
211211
if (format == null || !level.isGreaterOrEqual(logger.getEffectiveLevel())) {
212212
return FilterReply.NEUTRAL;

micrometer-core/src/main/java/io/micrometer/core/instrument/distribution/TimeWindowFixedBoundaryHistogram.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,10 @@ void outputSummary(PrintStream printStream, double bucketScaling) {
122122

123123
String bucketFormatString = "%14.1f %10d\n";
124124

125+
FixedBoundaryHistogram currentHistogram = currentHistogram();
125126
for (int i = 0; i < buckets.length; i++) {
126127
printStream.format(Locale.US, bucketFormatString, buckets[i] / bucketScaling,
127-
currentHistogram().values.get(i));
128+
currentHistogram.values.get(i));
128129
}
129130

130131
printStream.write('\n');

micrometer-core/src/main/java/io/micrometer/core/instrument/step/StepDistributionSummary.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public class StepDistributionSummary extends AbstractDistributionSummary impleme
4646
* Create a new {@code StepDistributionSummary}.
4747
* @param id ID
4848
* @param clock clock
49-
* @param distributionStatisticConfig distribution static configuration
49+
* @param distributionStatisticConfig distribution statistic configuration
5050
* @param scale scale
5151
* @param stepMillis step in milliseconds
5252
* @param supportsAggregablePercentiles whether it supports aggregable percentiles
@@ -60,7 +60,7 @@ public StepDistributionSummary(Id id, Clock clock, DistributionStatisticConfig d
6060

6161
@Override
6262
protected void recordNonNegative(double amount) {
63-
count.add(1);
63+
count.add(1L);
6464
total.add(amount);
6565
max.record(amount);
6666
}

micrometer-core/src/main/java/io/micrometer/core/instrument/step/StepTimer.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public StepTimer(final Id id, final Clock clock, final DistributionStatisticConf
5959
@Override
6060
protected void recordNonNegative(final long amount, final TimeUnit unit) {
6161
final long nanoAmount = (long) TimeUtils.convert(amount, unit, TimeUnit.NANOSECONDS);
62-
count.add(1);
62+
count.add(1L);
6363
total.add(nanoAmount);
6464
max.record(nanoAmount);
6565
}

0 commit comments

Comments
 (0)