Skip to content

fix(core): Restore directory processor interrupt flag - #5884

Open
0xadam-brown wants to merge 1 commit into
mainfrom
fix/directory-processor-thread-interrupt
Open

fix(core): Restore directory processor interrupt flag#5884
0xadam-brown wants to merge 1 commit into
mainfrom
fix/directory-processor-thread-interrupt

Conversation

@0xadam-brown

@0xadam-brown 0xadam-brown commented Aug 3, 2026

Copy link
Copy Markdown
Member

📜 Description

Restores thread interrupt status when the inter-envelope processing delay is interrupted so callers can observe the status after directory processing returns.

💡 Motivation and Context

Fixes JAVA-666

💚 How did you test it?

Via a new regression test

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

@linear-code

linear-code Bot commented Aug 3, 2026

Copy link
Copy Markdown

JAVA-666

Restore the interrupt status when the inter-envelope processing delay is interrupted so callers can observe the interruption after directory processing returns.

Add regression coverage for the interrupted delay path.

Fixes JAVA-666
Co-Authored-By: OpenCode <noreply@opencode.ai>
@0xadam-brown
0xadam-brown force-pushed the fix/directory-processor-thread-interrupt branch from 07a909c to 12a780d Compare August 3, 2026 14:03
@sentry

sentry Bot commented Aug 3, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.51.0 (1) release

⚙️ sentry-android Build Distribution Settings

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 311.87 ms 361.30 ms 49.43 ms
Size 0 B 0 B 0 B

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
0220a5c 452.20 ms 479.33 ms 27.12 ms
d15471f 310.26 ms 377.04 ms 66.78 ms
bbc35bb 298.53 ms 372.17 ms 73.64 ms
b8bd880 314.56 ms 336.50 ms 21.94 ms
62b579c 318.48 ms 367.71 ms 49.24 ms
d8b6ce1 314.62 ms 348.67 ms 34.05 ms
44472da 319.72 ms 351.90 ms 32.18 ms
e59e22a 368.02 ms 432.00 ms 63.98 ms
ee747ae 400.46 ms 423.61 ms 23.15 ms
462dea2 309.63 ms 352.22 ms 42.59 ms

App size

Revision Plain With Sentry Diff
0220a5c 0 B 0 B 0 B
d15471f 1.58 MiB 2.13 MiB 559.54 KiB
bbc35bb 1.58 MiB 2.12 MiB 553.01 KiB
b8bd880 1.58 MiB 2.29 MiB 722.92 KiB
62b579c 0 B 0 B 0 B
d8b6ce1 0 B 0 B 0 B
44472da 0 B 0 B 0 B
e59e22a 1.58 MiB 2.20 MiB 635.34 KiB
ee747ae 1.58 MiB 2.10 MiB 530.95 KiB
462dea2 0 B 0 B 0 B

@0xadam-brown 0xadam-brown added the sanity-check PR needs a lightweight review for obvious issues label Aug 3, 2026
@0xadam-brown
0xadam-brown marked this pull request as ready for review August 3, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sanity-check PR needs a lightweight review for obvious issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant