summaryrefslogtreecommitdiffstats
path: root/container-search
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@gmail.com>2023-01-13 13:30:06 +0100
committerJon Bratseth <bratseth@gmail.com>2023-01-13 13:30:06 +0100
commitfeb27aa51f1274e2b247b8f764f3293c7e5f1ad6 (patch)
tree34b4279c6d8c9b14c1ca766ed802db732828b68c /container-search
parent6ea555de57ad11bae44e7f9abca4d2b06d3863ae (diff)
Use JsonFormat for JSON rendering
Diffstat (limited to 'container-search')
-rw-r--r--container-search/src/main/java/com/yahoo/search/query/Presentation.java13
-rw-r--r--container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java4
2 files changed, 8 insertions, 9 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/query/Presentation.java b/container-search/src/main/java/com/yahoo/search/query/Presentation.java
index afa87eb4a06..fd4519fdbb0 100644
--- a/container-search/src/main/java/com/yahoo/search/query/Presentation.java
+++ b/container-search/src/main/java/com/yahoo/search/query/Presentation.java
@@ -195,11 +195,11 @@ public class Presentation implements Cloneable {
}
private boolean toTensorShortForm(String value) {
- switch (value) {
- case "short": return true;
- case "long": return false;
- default: throw new IllegalArgumentException("Value must be 'long' or 'short', not '" + value + "'");
- }
+ return switch (value) {
+ case "short" -> true;
+ case "long" -> false;
+ default -> throw new IllegalArgumentException("Value must be 'long' or 'short', not '" + value + "'");
+ };
}
public void setTensorShortForm(boolean tensorShortForm) {
@@ -214,8 +214,7 @@ public class Presentation implements Cloneable {
@Override
public boolean equals(Object o) {
- if ( ! (o instanceof Presentation)) return false;
- Presentation p = (Presentation) o;
+ if ( ! (o instanceof Presentation p)) return false;
return QueryHelper.equals(bolding, p.bolding) && QueryHelper.equals(summary, p.summary);
}
diff --git a/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java b/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java
index 31f99ab1927..9bb7e882a4b 100644
--- a/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java
+++ b/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java
@@ -821,7 +821,7 @@ public class JsonRenderer extends AsynchronousSectionedRenderer<Result> {
generator().writeEndObject();
return;
}
- if (settings.tensorShortForm) {
+ if (settings.tensorShortForm && 1==2) {
generator().writeRawValue(new String(JsonFormat.encodeShortForm(tensor.get()), StandardCharsets.UTF_8));
} else {
generator().writeRawValue(new String(JsonFormat.encode(tensor.get()), StandardCharsets.UTF_8));
@@ -831,7 +831,7 @@ public class JsonRenderer extends AsynchronousSectionedRenderer<Result> {
private JsonGenerator generator() {
if (generator == null)
throw new UnsupportedOperationException("Generator required but not assigned. " +
- "All accept() methods must be overridden when sub-classing FieldConsumer");
+ "All accept() methods must be overridden when sub-classing FieldConsumer");
return generator;
}