Remove references to jakarta from co.elastic.clients.json.JsonpMapper

Description

Hi folks,

I see that you rewrote Java client and instantly I have feature request for you.

I’d like references to jakarta to be removed from co.elastic.clients.json.JsonpMapper because current abstraction is not much of an abstraction really because it prevents me from implementing json mapper that uses something other than Jackson. In my case I’d love to use Moshi and I’m ready to contribute the implementation to you.

JsonMapper interface should be something like this:

import java.io.InputStream;
import java.lang.reflect.Type;

public interface JsonMapper {

    <T> T deserialize(Type type, String json);

    <T> T deserialize(Type type, InputStream source);

    <T> void serialize(Type type, T value);
}

Use of Type instead of Class<T> is intentional so that we can properly serialize generic value such as List<Map<String, Int>> whose type will be represented by ParameterizedTypeImpl.

I’m looking forward to your feedback, @swallez

[unoexperto]

We won’t change this decision

That’s ok 😃 I appreciate your time explaining your position to me!

[swallez]

Thanks! So I guess we can close this issue.