Hi Simon,
Makes sense to me to make that change - please do send in a patch.
Andy
On 14/01/2020 22:36, Fell, Simon wrote:
Hi,
I was looking at org.apache.jena.datatypes.TypeMapper and noticed that it’s a
shared instance, but it manages state in regular old HashMaps. Given that
getSafeTypeByName can end up calling registerDatatype, I don’t think its safe
to assume that these hashmaps are only written at construction/reset time.
Seems like these should be swapped to ConcurentHashMaps.
Thoughts? I’d be happy to put together a patch for this.
Cheers
Simon
