Consider Separate flow for Spans Events
@Litarnus is already working on this.
Since Nov 4, 2025.
Assessment
This issue has not been assessed yet.
Description
Instead of using applyToEvent for every type, we can split them by type.
This would help in a few ways:
- More clarity what each type does and what data it needs
- Prevents that data is accidentally attached to wrong types (e.g. tags on logs)
- We could have different Event types with their own data instead of one big class with the union of all fields
- (theoretical) performance gains by reducing the number of type checks and data copy operations
- Dominant language
- PHP
- Stars
- 1.9k
- Forks
- 474
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 8
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from getsentry/sentry-php
-
Improvement PHP Waiting for: Product Owner
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
getsentry/sentry-php#2173 · 1 comment ·
-
PHP Task
getsentry/sentry-php#2162 · 1 comment · 1 assignee ·
-
PHP
getsentry/sentry-php#2140 · 1 comment · 1 assignee ·
-
5.0 Improvement PHP
getsentry/sentry-php#1956 · 1 comment · 1 assignee ·
-
Improvement PHP Spans
getsentry/sentry-php#1953 · 1 comment · 1 assignee ·
All issues in getsentry/sentry-php
Similar issues
-
priority: p3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
googleapis/librarian#7636 ·
-
0. Needs triage bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
nextcloud/fulltextsearch#1011 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
phpstan/phpstan-doctrine#794 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Automattic/static-site-importer#1767 ·