Reuse Jackson JsonFactory, as it is thread safe

[skjolber]

Reuse Jackson JsonFactory.

Description

From JsonFactory Javadoc:

• Factory instances are thread-safe and reusable after configuration
• (if any). Typically applications and services use only a single
• globally shared factory instance, unless they need differently
• configured factories. Factory reuse is important if efficiency matters;
• most recycling of expensive construct is done on per-factory basis.

Motivation and Context

Improve performance.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

[skjolber]

Build breaks on

One or more dependencies were identified with known vulnerabilities in Logbook: JSON:
jackson-databind-2.9.9.jar (cpe:/a:fasterxml:jackson:2.9.9, cpe:/a:fasterxml:jackson-databind:2.9.9, com.fasterxml.jackson.core:jackson-databind:2.9.9) : CVE-2019-12814

As there is no 2.9.10 release yet; add to csv-suppresions, or wait?

[whiskeysierra]

As there is no 2.9.10 release yet; add to csv-suppresions, or wait?

Suppress it for now.

I'm still waiting for FasterXML/jackson-databind#2341

[whiskeysierra]

👍

[whiskeysierra]

Thanks for the contribution 🎉 and sorry that it took so long.