Skip to content

Conversation

@lev-serebryakov-jetbrains
Copy link
Collaborator

!!!WARNING!!! !!!WARNING!!! !!!WARNING!!!

This change CHANGES API but this change IS NOT ENFORCED statically. After this change all accesses to deserialized trace points MUST be wrapped into LazyTraceReader.inContext() or withTraceContext() calls or preceded by LazyTraceReader.switchGlobalContext() call (which is unsafe).

!!!WARNING!!! !!!WARNING!!! !!!WARNING!!!

@lev-serebryakov-jetbrains lev-serebryakov-jetbrains force-pushed the lev-serebryakov/switch-contexts-with-lazy-reader branch 2 times, most recently from 2e53c49 to 6779161 Compare August 29, 2025 13:10
!!!WARNING!!! !!!WARNING!!! !!!WARNING!!!

This change CHANGES API but this change IS NOT ENFORCED statically.
After this change all accesses to deserialized trace points MUST be wrapped into LazyTraceReader.inContext() or withTraceContext() calls or preceded by switchGlobalContext() call.

At least add `lazyReader.switchGlobalContext()` after `lazyReader.readRoots()`.

!!!WARNING!!! !!!WARNING!!! !!!WARNING!!!
@lev-serebryakov-jetbrains lev-serebryakov-jetbrains force-pushed the lev-serebryakov/switch-contexts-with-lazy-reader branch from 6779161 to c5dbe6a Compare September 6, 2025 17:37
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