rbeckman-nextgen / test-mc2

0 stars 0 forks source link

Fully implement java.util.concurrent.ConcurrentMap for global maps #4322

Open rbeckman-nextgen opened 5 years ago

rbeckman-nextgen commented 5 years ago

GlobalVariableStore and GlobalChannelVariableStore are both backed by ConcurrentHashMaps, but don't support any of the atomic methods from the ConcurrentMap interface for thread-safe operations.

Imported Issue. Original Details: Jira Issue Key: MIRTH-4468 Reporter: agermano Created: 2019-10-09T22:24:55.000-0700

rbeckman-nextgen commented 5 years ago

Those "sync" methods were created a long time ago, probably to satisfy some narrow use-case at the time. We need to overhaul those to make them easier to understand and more applicable to real use-cases around synchronization. It's probably fine to just rip out the old stuff without deprecating anything, since I doubt anyone uses them.

Imported Comment. Original Details: Author: narupley Created: 2019-10-29T12:40:10.000-0700