Skip to content

Commit 05a18ce

Browse files
committed
chore(model): Only log edges to remove at info level
Avoid computing them always and logging as a warning. Signed-off-by: Sebastian Schuberth <[email protected]>
1 parent ac35731 commit 05a18ce

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

model/src/main/kotlin/utils/DependencyGraphBuilder.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -191,10 +191,13 @@ class DependencyGraphBuilder<D>(
191191
private fun Set<DependencyGraphEdge>.removeCycles(): Set<DependencyGraphEdge> {
192192
val edges = toMutableSet()
193193
val edgesToKeep = breakCycles(edges)
194-
val edgesToRemove = edges - edgesToKeep
195194

196-
edgesToRemove.forEach {
197-
logger.warn { "Removing edge '${it.from} -> ${it.to}' to break a cycle." }
195+
if (logger.delegate.isInfoEnabled) {
196+
val edgesToRemove = edges - edgesToKeep
197+
198+
edgesToRemove.forEach {
199+
logger.info { "Removing edge '${it.from} -> ${it.to}' to break a cycle." }
200+
}
198201
}
199202

200203
return filterTo(mutableSetOf()) { it in edgesToKeep }

0 commit comments

Comments
 (0)