profiler.id is always attached, even if ProfilingManager is rate-limiting
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 54/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- java
- Bereich
- mobile, performance
Rechercherichtung
Beginne in sentry/src/main/java/io/sentry/SentryTracer.java bei der verknüpften Trace-/Profil-Zuordnung und verfolge dann ProfilingManager und PerfettoProfiler.start() bis durch processResult. Bestätige, wie rate-limitierte Profiling-Ergebnisse gemeldet werden, und überprüfe, dass Traces keine profiler.id erhalten, wenn das entsprechende Profil fehlt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
This has been observed for Perfetto Profiling in our demo app, where a trace had a profiler.id set, but the actual profile was missing.
When linking traces and profiles, we link them unconditionally, e.g. here: https://github.com/getsentry/sentry-java/blob/bcd3e76a8033a30cf291fc84c726d40205b21db2/sentry/src/main/java/io/sentry/SentryTracer.java#L552
This stems from ProfilingManager, where requests can be rate-limited, and rejections are only reported via async profiling result. Or in other words PerfettoProfiler.start() will always return true, and only processResult at a later point will be able to detect rate limiting.
- Vorherrschende Sprache
- Kotlin
- Sterne
- 1.4k
- Forks
- 478
- Ø Merge
- 2 T. 20 Std.
- Gemergte PRs (30 T.)
- 71
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus getsentry/sentry-java
-
Improvement Java Platform: Android Platform: Java
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
getsentry/sentry-java#6145 · 1 Kommentar · 1 zugewiesene Person ·
-
Bug Java Platform: Android Platform: Java
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
getsentry/sentry-java#6138 · 1 Kommentar ·
-
Feature Java Platform: Java Spans
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
getsentry/sentry-java#5984 · 1 Kommentar ·
-
Android Task Traces
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
getsentry/sentry-java#5376 · 1 Kommentar ·
-
Android Docs Errors
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 64/100
getsentry/sentry-java#5375 · 1 Kommentar ·
Alle Issues in getsentry/sentry-java
Ähnliche Issues
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
libre-tube/LibreTube#8803 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Telegram !!! Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Xposed-Modules-Repo/com.fuck.iab#24 · 3 Kommentare · 1 Reaktion ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
SimonHalvdansson/Harmonic-HN#361 ·