SentryLogcatAdaptor consumes unnecessary time on main thread

[kozaxinan]

Integration

sentry-android

Build System

Gradle

AGP Version

8.4.0

Proguard

Enabled

Version

7.8.0

Steps to Reproduce

Log a warning or error.
SentryLogcatAdapter calls addAsBreadcrumb.

Expected Result

It should not take more time than calling log without Sentry.

Actual Result

Gson operations takes a more than 3ms for every log operation.
That quickly aggregates for an app that has lots of log calls.

cpu-art-20240517T150036.trace.zip

┆Issue is synchronized with this Jira Improvement by Unito

[kozaxinan]

I come up with following solution.
I wonder if it would be better to disable NDK sync.

 val ndkScope = scopeObservers.firstOrNull { it is NdkScopeObserver }

                scopeObservers.apply {
                    remove(ndkScope)
                }

                if (ndkScope != null) {
                    addScopeObserver(
                        object : IScopeObserver by ndkScope {

                            override fun addBreadcrumb(crumb: Breadcrumb) {
                                coroutineScope.launch {
                                    ndkScope.addBreadcrumb(crumb)
                                }
                            }
                        }
                    )
                }

[stefanosiano]

hi @kozaxinan, thanks for the report!
We'll have a look at it soon!
Probably related to #3046

[romtsn]

@kozaxinan we're going to close this as a duplicate of #3046, and reprioritize that one to tackle it sooner. Thanks for investigating this and providing us with data, please track it in the other issue 🙏