Silicon-Age / opal

Silicon Age's ORM layer.
0 stars 0 forks source link

Investigate replacing OpalCaches with ConcurrentMap #8

Open jonahgreenthal opened 1 year ago

jonahgreenthal commented 1 year ago

[migrated from Phabricator]

Brian Goetz extols the virtues of ConcurrentMaps in his book, and the new computeIfAbsent function seems like it would make it very easy to swap this in for the OpalCaches.

Superficially, [R. thinks] this could cut down on the chance of deadlocks and maybe improve performance, but [he]'d have to think harder about the possible race conditions.