getsentry / getsentry/sentry-java

New Performance API Wishlist

Aperta
#4,215 4 commenti 2 reazioni 0 assegnatari Vedi su GitHub
Android Feature Spans
Lingua principale
Kotlin
Stelle
1.4k
Fork
478
Merge medio
2g 23h
PR unite (30g)
67

Descrizione

### Description

This issue shall serve as a sort of wishlist where we can collect ideas for the new Performance API.

If you have anything you'd like us to improve when reworking our Performance API in the Java SDK, please do leave a comment.

Here's some ideas:
- Get rid of transaction, only have spans in the API (note, depending on when we rewrite the API, the Sentry UI in product will likely still have transactions but the Java SDK API will not have those anymore)
- Have spans added to the scope by default (this may not apply to Android due to `globalHubMode`)
- Make it easier to continue a trace (`Sentry.continueTrace` -> `Sentry.startTransaction` is more complicated than it needs to be)
- Consider moving to something like a Propagator design that OpenTelemetry has for extracting and injecting tracing information
- Merge different objects used for tracing information (`PropagationContext`, `TransactionContext`/`SpanContext`, `Baggage`, `TraceContext`)
- Have the span implement `AutoClosable` so it can be used in `try` with resources statements and have the span finished automatically
- Align attributes / data closer to OpenTelemetry (this may mean limited support for data types)

NOTE: A reworked Performance API will likely be a breaking change where we completely remove the old API as it will likely be very hard (if not impossible) to make them coexist. So we can use this as an opportunity to clean up.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.