Skip to content

Comments

use ConcurrentHashMap for play.classloading.ApplicationClassloader#as…#57

Merged
renat-nosto merged 1 commit intomasterfrom
improvement/use_ConcurrentHashMap_for_play.classloading.ApplicationClassloader#assignableClassesByName
Nov 4, 2022
Merged

use ConcurrentHashMap for play.classloading.ApplicationClassloader#as…#57
renat-nosto merged 1 commit intomasterfrom
improvement/use_ConcurrentHashMap_for_play.classloading.ApplicationClassloader#assignableClassesByName

Conversation

@renat-nosto
Copy link
Member

@renat-nosto renat-nosto commented Nov 3, 2022

…signableClassesByName

Pull Request Checklist

Helpful things

Fixes

an attempt to fix https://prod-graylog1.us-east-1.nos.to/search?q=exceptionClass%3A%22java.util.ConcurrentModificationException%22&rangetype=relative&relative=432000
This map (cache) fails upon application startup

Purpose

What does this PR do?

Background Context

Why did you take this approach?

References

Are there any relevant issues / PRs / mailing lists discussions?

Copy link

@leanish leanish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 💪🏼

@renat-nosto renat-nosto merged commit 20e7d2e into master Nov 4, 2022
@renat-nosto renat-nosto deleted the improvement/use_ConcurrentHashMap_for_play.classloading.ApplicationClassloader#assignableClassesByName branch November 4, 2022 13:04
@renat-nosto renat-nosto restored the improvement/use_ConcurrentHashMap_for_play.classloading.ApplicationClassloader#assignableClassesByName branch January 7, 2026 14:33
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