Skip to content

feat(core): [Cache Tracing 2] Add enableCacheTracing option#5163

Merged
adinauer merged 1 commit intofeat/cache-tracingfrom
feat/cache-tracing-option
Mar 9, 2026
Merged

feat(core): [Cache Tracing 2] Add enableCacheTracing option#5163
adinauer merged 1 commit intofeat/cache-tracingfrom
feat/cache-tracing-option

Conversation

@adinauer
Copy link
Member

@adinauer adinauer commented Mar 5, 2026

PR Stack (Cache Tracing)

  • #5162 — Add SentryCacheWrapper and SentryCacheManagerWrapper
  • #5163 — Add enableCacheTracing option
  • #5164 — Add BeanPostProcessor and auto-configuration
  • #5167 — Add cache tracing e2e sample

📜 Description

Adds enableCacheTracing boolean option to SentryOptions and ExternalOptions, gating the cache wrapper span creation behind an opt-in property (sentry.enable-cache-tracing). Default is false.

💚 How did you test it?

  • SentryOptionsTest — default value, merge behavior
  • ExternalOptionsTest — property binding (true/false)
  • SentryCacheWrapperTest — no span created when option disabled

📝 Checklist

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • No breaking change or entry added to the changelog.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Core

  • [Cache Tracing 2] Add enableCacheTracing option by adinauer in #5163
  • [Cache Tracing 2] Add enableCacheTracing option by adinauer in #5163

Spring

  • Cache Tracing by adinauer in #5165
  • [Cache Tracing 3] Add BeanPostProcessor and auto-configuration by adinauer in #5164
  • [Cache Tracing 1] Add SentryCacheWrapper and SentryCacheManagerWrapper by adinauer in #5162
  • [Cache Tracing 1] Add SentryCacheWrapper and SentryCacheManagerWrapper by adinauer in #5137

Bug Fixes 🐛

  • (screenshot) Add dontwarn replay rules to sentry-android-core by romtsn in #5153
  • (sessions) Finalize previous session even when auto session tracking is disabled by romtsn in #5154

🤖 This preview updates automatically when you update the PR.

@adinauer
Copy link
Member Author

adinauer commented Mar 5, 2026

PR Stack (Cache Tracing)

  • #5161Collection PR (full diff → main)
  • #5162 — Add SentryCacheWrapper and SentryCacheManagerWrapper
  • #5163 — Add enableCacheTracing option
  • #5164 — Add BeanPostProcessor and auto-configuration

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- [Cache Tracing 2] Add enableCacheTracing option ([#5163](https://github.com/getsentry/sentry-java/pull/5163))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against 5e081fc

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 312.72 ms 356.96 ms 44.24 ms
Size 0 B 0 B 0 B

Baseline results on branch: feat/cache-tracing-wrappers

Startup times

Revision Plain With Sentry Diff
2876c40 339.24 ms 405.65 ms 66.41 ms
8baac33 313.15 ms 367.84 ms 54.69 ms
3e5f591 317.48 ms 356.80 ms 39.32 ms

App size

Revision Plain With Sentry Diff
2876c40 1.58 MiB 2.29 MiB 723.29 KiB
8baac33 0 B 0 B 0 B
3e5f591 0 B 0 B 0 B

Previous results on branch: feat/cache-tracing-option

Startup times

Revision Plain With Sentry Diff
e1f4f5b 309.67 ms 365.26 ms 55.59 ms
6dcba10 285.09 ms 372.84 ms 87.75 ms

App size

Revision Plain With Sentry Diff
e1f4f5b 0 B 0 B 0 B
6dcba10 1.58 MiB 2.29 MiB 722.92 KiB

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@adinauer adinauer force-pushed the feat/cache-tracing-option branch from d137d4e to 5e081fc Compare March 9, 2026 08:06
Base automatically changed from feat/cache-tracing-wrappers to feat/cache-tracing March 9, 2026 08:07
@adinauer adinauer merged commit 5e081fc into feat/cache-tracing Mar 9, 2026
55 of 56 checks passed
@adinauer adinauer deleted the feat/cache-tracing-option branch March 9, 2026 08:07
@sentry
Copy link

sentry bot commented Mar 9, 2026

Sentry Build Distribution

App Version Configuration
SDK Size 8.34.0 (1) release

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.

1 participant