Skip to content

fix(tracing): Make sure idle transaction does not override other transactions#7725

Merged
AbhiPrasad merged 1 commit intodevelopfrom
abhi-idle-transaction-scope
Apr 4, 2023
Merged

fix(tracing): Make sure idle transaction does not override other transactions#7725
AbhiPrasad merged 1 commit intodevelopfrom
abhi-idle-transaction-scope

Conversation

@AbhiPrasad
Copy link
Copy Markdown
Contributor

See: #7660

Make sure that if another transaction is set on the scope while an idle transaction is occuring, don't forcefully remove that transaction from the scope on idle transaction finish.

@AbhiPrasad AbhiPrasad requested review from a team, Lms24 and mydea and removed request for a team April 4, 2023 11:10
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 4, 2023

size-limit report 📦

Path Size
@sentry/browser - ES5 CDN Bundle (gzipped + minified) 20.73 KB (+0.01% 🔺)
@sentry/browser - ES5 CDN Bundle (minified) 64.75 KB (0%)
@sentry/browser - ES6 CDN Bundle (gzipped + minified) 19.28 KB (0%)
@sentry/browser - ES6 CDN Bundle (minified) 57.23 KB (0%)
@sentry/browser - Webpack (gzipped + minified) 20.84 KB (0%)
@sentry/browser - Webpack (minified) 68.2 KB (0%)
@sentry/react - Webpack (gzipped + minified) 20.87 KB (0%)
@sentry/nextjs Client - Webpack (gzipped + minified) 48.69 KB (-0.01% 🔽)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified) 28.31 KB (-0.03% 🔽)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified) 26.53 KB (-0.02% 🔽)
@sentry/replay ES6 CDN Bundle (gzipped + minified) 44.88 KB (+0.01% 🔺)
@sentry/replay - Webpack (gzipped + minified) 38.86 KB (0%)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified) 63.58 KB (-0.02% 🔽)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified) 56.62 KB (0%)

@AbhiPrasad AbhiPrasad merged commit 95e51a2 into develop Apr 4, 2023
@AbhiPrasad AbhiPrasad deleted the abhi-idle-transaction-scope branch April 4, 2023 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants