Abuse of new ObjectMapper()

[xuyang2]

Found 100+ occurrences: new ObjectMapper()
ObjectMapper is heavy-weight and thread safe. It should be resused.

according to JacksonFAQ:

Is ObjectMapper thread-safe?
Short answer: yes
Long answer: yes, as long as you always configure instance before use, and do not call configure methods during operation (or synchronize such calls appropriately). Usually it is better to construct separate mapper instance if configurations differ in any case.
Further: it is beneficial to use just one instance (or small number of instances) for data binding; many optimizations for reuse (of symbol tables, some buffers) depend on ObjectMapper instances being reused.
For more information, check out JacksonFAQThreadSafety

[kishoreg]

Thanks, do you have a patch?.

[xuyang2]

Sorry. I did not notice that you are using jackson1 introduced by avro.
The Jackson wiki is mainly about jackson2.
The jackson1 ObjectMapper seems to be thread safe, but I'm not sure.

[gortiz]

Given #8736 was merged, I think we can close this issue.