Prevent race condition issues when parsing types #1352
+11
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Long story short - sourcery is crashing sometimes on processing files in our product
This is mostly because of the multithreaded access to the
ParserResultsComposed
which is responsible for resolving types.Since that class has a somewhat cache (when using / creating
actualTypeName
s and checking if they're there), this can lead and leads to the situations, whenTypeName
being overreleasedThis is more hack than fix since technically, this makes type resolving almost serial.
In any case:
While, this is, probably, not the best solution, PR is here, so this can be a hack.
P.S. Alternative is to turn on
--serial
option.