Merge pull request #1462 from akka/wip-slight-improvement-to-coroner-output-√
Minor improvement to Coroner thread count formatting.
This commit is contained in:
commit
ba88c44b6f
2 changed files with 4 additions and 4 deletions
|
|
@ -89,7 +89,7 @@ object Coroner {
|
||||||
val startThreads = threadMx.getThreadCount
|
val startThreads = threadMx.getThreadCount
|
||||||
if (displayThreadCounts) {
|
if (displayThreadCounts) {
|
||||||
threadMx.resetPeakThreadCount()
|
threadMx.resetPeakThreadCount()
|
||||||
out.println(s"Coroner Thread Count Start: current = ${startThreads} in ${reportTitle}")
|
out.println(s"Coroner Thread Count starts at $startThreads in $reportTitle")
|
||||||
}
|
}
|
||||||
watchedHandle.started()
|
watchedHandle.started()
|
||||||
try {
|
try {
|
||||||
|
|
@ -106,7 +106,7 @@ object Coroner {
|
||||||
} finally {
|
} finally {
|
||||||
if (displayThreadCounts) {
|
if (displayThreadCounts) {
|
||||||
val endThreads = threadMx.getThreadCount
|
val endThreads = threadMx.getThreadCount
|
||||||
out.println(s"Coroner Thread Count End: current = ${endThreads}, diff = ${endThreads - startThreads}, peak = ${threadMx.getPeakThreadCount} in ${reportTitle}")
|
out.println(s"Coroner Thread Count started at $startThreads, ended at $endThreads, peaked at ${threadMx.getPeakThreadCount} in $reportTitle")
|
||||||
}
|
}
|
||||||
out.flush()
|
out.flush()
|
||||||
watchedHandle.finished()
|
watchedHandle.finished()
|
||||||
|
|
|
||||||
|
|
@ -49,8 +49,8 @@ class CoronerSpec extends WordSpec with MustMatchers {
|
||||||
coroner.cancel()
|
coroner.cancel()
|
||||||
Await.ready(coroner, 1.second)
|
Await.ready(coroner, 1.second)
|
||||||
})
|
})
|
||||||
report must include("Coroner Thread Count Start:")
|
report must include("Coroner Thread Count starts at ")
|
||||||
report must include("Coroner Thread Count End:")
|
report must include("Coroner Thread Count started at ")
|
||||||
report must include("XXXX")
|
report must include("XXXX")
|
||||||
report must not include ("Coroner's Report")
|
report must not include ("Coroner's Report")
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue