Example from the readme doc fails with Jackson, works with Yasson about elasticsearch-java HOT 4 CLOSED

Addition: When I change to the Yasson jsonb processing by using this:

Transport transport = new RestClientTransport(restClient, new JsonbJsonpMapper());

instead of the JacksonJsonpMapper() the call succeeds.

from elasticsearch-java.

Thanks for trying out the new Java client! Still early days, but feedback is very welcome!

There's an issue in the JacksonJsonpParser class that converts Jackson parser events into JSON-P events. I'm currently working on it, and will update once I've fixed it.

from elasticsearch-java.

Looks great up to now. I wrote reactive counterparts to ApiClient, Transport and ElasticsearchClient (the last one just for the first search method) and can reuse the Endpoints and part of the code from the RestClientTransport to send the requests and parse the responses over the wire for imperative and reactive code. That will definitely allow me to clean up the codebase in Spring Data Elasticsearch when we use this for reactive and imperative code.
My biggest problem seems to be that I already have a class named ReactiveElasticsearchClient and I need to support them both until version 8 of ES.

from elasticsearch-java.

Fixed in 3bef3fd that also adds tests. I've pushed a new snapshot that also includes your changes from #6. Thanks again!

from elasticsearch-java.