-
Notifications
You must be signed in to change notification settings - Fork 260
Possible bug building SortOptions object from json #573
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
Category: Bug
Something isn't working
Comments
Hello, thank you for the report! Definitely a bug, we'll investigate this. |
For new guests to this issue, this comment may help you resolve this ancient issue temporarily 😂 I've figured out a possible solution by copying and override some source code:
Code example: import co.elastic.clients.elasticsearch._types.SortOptions;
import co.elastic.clients.json.JsonpMapper;
import com.fasterxml.jackson.databind.ObjectMapper;
import jakarta.json.stream.JsonParser;
import java.io.StringReader;
import java.util.Map;
public class DummySortOptionBuilder extends SortOptions.Builder {
@Override
public SortOptions.Builder withJson(JsonParser parser, JsonpMapper mapper) {
var option = SortOptions._DESERIALIZER.deserialize(parser, mapper);
if (option.isScore()) {
score(option.score());
} else if (option.isDoc()) {
doc(option.doc());
} else if (option.isGeoDistance()) {
geoDistance(option.geoDistance());
} else if (option.isScript()) {
script(option.script());
} else if (option.isField()) {
field(option.field());
}
return this;
}
public static SortOptions fromJson(String json) {
return new DummySortOptionBuilder()
.withJson(new StringReader(json))
.build();
}
public static void main(String[] args) throws Exception {
var mapper = new ObjectMapper();
// sort by score
System.out.println(fromJson(mapper.writeValueAsString("_score")));
System.out.println(fromJson("\"_score\""));
// sort by doc
System.out.println(fromJson(mapper.writeValueAsString("_doc")));
System.out.println(fromJson("\"_doc\""));
// sort by field
System.out.println(fromJson(mapper.writeValueAsString("term")));
System.out.println(fromJson("\"term\""));
// sort by field with options
System.out.println(fromJson(mapper.writeValueAsString(Map.of("term", "asc"))));
System.out.println(fromJson("{\"term\" : \"asc\"}"));
}
} |
More directly, the workaround would be to avoid these two forms:
and instead, construct the JSON with the most verbose form:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Java API client version
8.7.1
Java version
19.0.1
Elasticsearch Version
8.4.3
Problem description
I'm trying to create a SortOptions object for an elastic search query using the Create API objects from JSON data method. But I keep getting the following exception:
Code sample:
I've also tried setting test to:
And got the same results. I'll admit the 'bug' may just be in my simple minded interpretation of what subsection/format of json input is required to get this to work 😓, on the plus side that would be an easy fix.
The text was updated successfully, but these errors were encountered: