MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2025-12-15 02:59:33 +07:00
Code Issues Packages Projects Releases Wiki Activity

IJPL-160781 DevServer sometimes takes 4+ minutes to build plugin

Browse Source 
Let's try to remove all additional and not essential logging.
Hypothesis: We might output too much messages to TC which leads to buffers overflow and the total freezes of tests.

GitOrigin-RevId: a58bd9cc890781f2bf381dd4bff24163d9dc1cc5
This commit is contained in:
Maxim.Kolmakov
2024-09-04 22:14:18 +02:00
committed by intellij-monorepo-bot
parent 36a5960b32
commit 901b99b569
1 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
build/tasks/src/org/jetbrains/intellij/build/telemetry/span.kt
View File

@@ -9,7 +9,6 @@ import io.opentelemetry.extension.kotlin.asContextElement
import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.withContext
import org.jetbrains.annotations.ApiStatus.Internal
import org.jetbrains.intellij.build.logging.TeamCityBuildMessageLogger
import org.jetbrains.intellij.build.telemetry.TraceManager.spanBuilder
import kotlin.coroutines.CoroutineContext
import kotlin.coroutines.EmptyCoroutineContext
@@ -28,10 +27,8 @@ suspend fun <T> SpanBuilder.block(
TraceManager.scheduleExportPendingSpans()
return startSpan().useWithoutActiveScope { span ->
// see `use` below why `withContext` must be inner
TeamCityBuildMessageLogger.withBlock(span) {
withContext(Context.current().with(span).asContextElement() + coroutineContext) {
operation(span)
}
withContext(Context.current().with(span).asContextElement() + coroutineContext) {
operation(span)
}
}
}
@@ -47,10 +44,8 @@ suspend inline fun <T> SpanBuilder.use(
return startSpan().useWithoutActiveScope { span ->
// inner `withContext` to ensure that we report the end of the span only when all child tasks are completed,
// the same for `withFlow` - must be out of `withContext`
TeamCityBuildMessageLogger.withFlow(span) {
withContext(Context.current().with(span).asContextElement() + context) {
withContext(Context.current().with(span).asContextElement() + context) {
operation(span)
}
}
}
}