Skip to content

Conversation

ThomasKrieger
Copy link

Resolves #4981

replaced map.containsKey with get and != null in MapStepRegistry
And test with vmlens to test the race condition

@ThomasKrieger ThomasKrieger force-pushed the fix/NullPointerException-parallel-remove-MapStepRegistry branch from 8d90e13 to afa57b1 Compare September 24, 2025 06:52
@fmbenhassine
Copy link
Contributor

Hi @ThomasKrieger ,

Thank you for opening this PR! MapStepRegistry is now deprecated for removal and we are not planning to put any effort in maintaining it anymore. Therefore, I am going to close this PR.

However, if you manage to find other issues thanks to vmlens, you are welcome to contribute other PRs. Please note that we have no plan to add vmlens as a dependency and use its maven plugin, but we are open to merge any bug-fix that it may detect.

Keep the good work up on vmlens!

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.

Race Condition MapStepRegistry unregisterStepsFromJob and getStep

2 participants