Replace copied mjson with a standard library that is already provided by runtimes - org.eclipse.parsson:parsson; resolve #288

[rhusar]

No description provided.

[rhusar]

@davido Any chance you would be able to review this?

[davido]

I’m not a committer on this project, so I can’t formally review or approve this change.

While migrating one of our Gerrit plugins to Bazel Modules, that depends on jgroups and jgroups-kubernetes I was tracking the dependencies that get shaded into the high-availability plugin. During that process, I noticed that some unrelated artifacts are being shaded but are not declared in the corresponding pom.xml.

See this change:

https://gerrit-review.googlesource.com/c/plugins/high-availability/+/561581/6/high-availability_third_party_runtime_jars.allowlist.txt

[rhusar]

@davido This is OSS so you can review anything on GH :-) But yes, the review doesn't show as green check mark but only grey one.

Anyway, this was missing try-with-resources blocks in the Json.createReader(..) fixed now.

[rhusar]

I would appreciate if you could review @pferraro as this is the last change required for the release so we can include this in WF 40 Beta.

[rhusar]

Thanks @ehsavoie for the review MUCH appreciated!

The feedback about code style discussed, I will fix in a subsequent PR.