search

apache / celeborn

Apache Celeborn is an elastic and high-performance service for shuffle and spilled data.
https://celeborn.apache.org/
Apache License 2.0
893 stars 361 forks source link

[CELEBORN-1619][CELEBORN-474][FOLLOWUP] TagsManager uses JavaUtils#newConcurrentHashMap to speed up ConcurrentHashMap#computeIfAbsent #2922

Closed SteNicholas closed 6 days ago

SteNicholas commented 1 week ago

What changes were proposed in this pull request?

TagsManager uses JavaUtils#newConcurrentHashMap to speed up ConcurrentHashMap#computeIfAbsent.

Follow up #2844.

Why are the changes needed?

Celeborn supports JDK8, which could meet the bug mentioned in JDK-8161372. Therefore, it's better to use JavaUtils#newConcurrentHashMap to speed up ConcurrentHashMap#computeIfAbsent.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI.

SteNicholas commented 1 week ago

Ping @turboFei.

SteNicholas commented 6 days ago

Merged to main(v0.6.0).