Rework synchronization in `ProtobufMapper`

FasterXML / jackson-dataformats-binary

Uber-project for standard Jackson binary format backends: avro, cbor, ion, protobuf, smile

Apache License 2.0

310 stars 133 forks source link

Closed pjfanning closed 5 months ago

pjfanning commented 6 months ago

also marked the variable as volatile as this is something that is often done when you have concurrent access
the new code tries a little harder not to create multiple DescriptorLoaders. The old code could have queued up multiple threads where l was null and then each got the synchronization lock in turn and then created a fresh DescriptorLoader.

pjfanning commented 5 months ago

@cowtowncoder would you have time to review this?

cowtowncoder commented 5 months ago

@cowtowncoder would you have time to review this?

Will do, thank you for reminder.