Skip to content

Commit

Permalink
Custom toString() for Pausers to be descriptive, Fixes #251
Browse files Browse the repository at this point in the history
  • Loading branch information
peter-lawrey committed May 13, 2024
1 parent 54b547f commit 308c732
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/main/java/net/openhft/chronicle/threads/LongPauser.java
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@
* progressively increasing the pause time from a minimum to a specified maximum duration.
*/
public class LongPauser implements Pauser, TimingPauser {
private static final long MAX_FACTOR = Jvm.isDebug() ? 100 : 1;
private static final String SHOW_PAUSES = Jvm.getProperty("pauses.show");
private final long minPauseTimeNS;
private final long maxPauseTimeNS;
Expand Down Expand Up @@ -68,7 +67,7 @@ public LongPauser(int minBusy, int minYield, long minTime, long maxTime, @NotNul
this.minBusyNS = timeUnit.toNanos(minBusy);
this.minYieldNS = timeUnit.toNanos(minYield);
this.minPauseTimeNS = timeUnit.toNanos(minTime);
this.maxPauseTimeNS = timeUnit.toNanos(maxTime) * MAX_FACTOR;
this.maxPauseTimeNS = timeUnit.toNanos(maxTime);
pauseTimeNS = minPauseTimeNS;
}

Expand Down Expand Up @@ -237,14 +236,14 @@ public String toString() {
if (maxPauseTimeNS == balancedSample.maxPauseTimeNS) {
return "PauserMode.balanced";
} else {
return "Pauser.balancedUpToMillis(" + maxPauseTimeNS / MAX_FACTOR / 1_000_000 + ")";
return "Pauser.balancedUpToMillis(" + maxPauseTimeNS / 1_000_000 + ")";
}
}
if (minBusyNS == millisSample.minBusyNS
&& minYieldNS == millisSample.minYieldNS
&& minPauseTimeNS >= millisSample.minPauseTimeNS
&& maxPauseTimeNS >= millisSample.maxPauseTimeNS)
return "Pauser.milli(" + minPauseTimeNS / 1_000_000 + ", " + maxPauseTimeNS / MAX_FACTOR / 1_000_000 + ")";
return "Pauser.milli(" + minPauseTimeNS / 1_000_000 + ", " + maxPauseTimeNS / 1_000_000 + ")";

if (minBusyNS == sleepySample.minBusyNS
&& minYieldNS == sleepySample.minYieldNS
Expand Down

0 comments on commit 308c732

Please sign in to comment.