diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-04-17 16:54:27 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2023-04-17 16:54:27 +0200 |
commit | 4d4e342c306b3db335d38067b1f4d9314e48ea94 (patch) | |
tree | 0431e666081872ef464a66859c0ef53050759b37 /container-search/src/main/java/com/yahoo | |
parent | ed8d16fb6ca6cfae34fc4d399038306e721a07f0 (diff) |
Render raw values in grouping as base64.
Diffstat (limited to 'container-search/src/main/java/com/yahoo')
3 files changed, 14 insertions, 9 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/grouping/result/RawBucketId.java b/container-search/src/main/java/com/yahoo/search/grouping/result/RawBucketId.java index 9576f548f4a..cab1b1928c4 100644 --- a/container-search/src/main/java/com/yahoo/search/grouping/result/RawBucketId.java +++ b/container-search/src/main/java/com/yahoo/search/grouping/result/RawBucketId.java @@ -2,6 +2,7 @@ package com.yahoo.search.grouping.result; import java.util.Arrays; +import java.util.Base64; /** * This class is used in {@link Group} instances where the identifying @@ -18,6 +19,6 @@ public class RawBucketId extends BucketGroupId<byte[]> { * @param to The identifying exclusive-to raw buffer. */ public RawBucketId(byte[] from, byte[] to) { - super("raw_bucket", from, Arrays.toString(from), to, Arrays.toString(to)); + super("raw_bucket", from, Base64.getEncoder().encodeToString(from), to, Base64.getEncoder().encodeToString(to)); } } diff --git a/container-search/src/main/java/com/yahoo/search/grouping/result/RawId.java b/container-search/src/main/java/com/yahoo/search/grouping/result/RawId.java index de711d0c218..b9c18aea380 100644 --- a/container-search/src/main/java/com/yahoo/search/grouping/result/RawId.java +++ b/container-search/src/main/java/com/yahoo/search/grouping/result/RawId.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.search.grouping.result; -import java.util.Arrays; +import java.util.Base64; /** * This class is used in {@link Group} instances where the identifying expression evaluated to a {@link Byte} array. @@ -16,6 +16,6 @@ public class RawId extends ValueGroupId<byte[]> { * @param value The identifying byte array. */ public RawId(byte[] value) { - super("raw", value, Arrays.toString(value)); + super("raw", value, Base64.getEncoder().encodeToString(value)); } } 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 b36c8788877..e15a1116175 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 @@ -57,7 +57,7 @@ import java.math.BigDecimal; import java.math.BigInteger; import java.nio.charset.StandardCharsets; import java.util.ArrayDeque; -import java.util.Arrays; +import java.util.Base64; import java.util.Deque; import java.util.Map; import java.util.Optional; @@ -420,9 +420,9 @@ public class JsonRenderer extends AsynchronousSectionedRenderer<Result> { } protected void renderGroupMetadata(GroupId id) throws IOException { - if (!(id instanceof ValueGroupId || id instanceof BucketGroupId)) return; + if (!(id instanceof ValueGroupId<?> || id instanceof BucketGroupId)) return; - if (id instanceof ValueGroupId valueId) { + if (id instanceof ValueGroupId<?> valueId) { generator.writeStringField(GROUPING_VALUE, getIdValue(valueId)); } else { BucketGroupId<?> bucketId = (BucketGroupId<?>) id; @@ -434,15 +434,19 @@ public class JsonRenderer extends AsynchronousSectionedRenderer<Result> { } private static String getIdValue(ValueGroupId<?> id) { - return (id instanceof RawId ? Arrays.toString(((RawId) id).getValue()) : id.getValue()).toString(); + return (id instanceof RawId ? id : id.getValue()).toString(); } private static String getBucketFrom(BucketGroupId<?> id) { - return (id instanceof RawBucketId ? Arrays.toString(((RawBucketId) id).getFrom()) : id.getFrom()).toString(); + if (id instanceof RawBucketId rawBucketId) + return Base64.getEncoder().encodeToString(rawBucketId.getFrom()); + return id.getFrom().toString(); } private static String getBucketTo(BucketGroupId<?> id) { - return (id instanceof RawBucketId ? Arrays.toString(((RawBucketId) id).getTo()) : id.getTo()).toString(); + if (id instanceof RawBucketId rawBucketId) + return Base64.getEncoder().encodeToString(rawBucketId.getTo()); + return id.getTo().toString(); } protected void renderTotalHitCount(Hit hit) throws IOException { |