diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2019-06-01 16:28:18 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2019-06-01 16:28:18 +0200 |
commit | aa8f671cfc7a6ab990841fafba3207a6f5181a0f (patch) | |
tree | 9e36e78928bcf28ecc832a6cc5cc0a3a8a7956a0 /container-search/src/main/java/com/yahoo/fs4 | |
parent | 443437a83cd1c3b4d55c732e8756d5c0b1595902 (diff) |
Keep the checks in one place
Diffstat (limited to 'container-search/src/main/java/com/yahoo/fs4')
-rw-r--r-- | container-search/src/main/java/com/yahoo/fs4/MapEncoder.java | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/container-search/src/main/java/com/yahoo/fs4/MapEncoder.java b/container-search/src/main/java/com/yahoo/fs4/MapEncoder.java index 3e45d7e5f53..f7e01bc1254 100644 --- a/container-search/src/main/java/com/yahoo/fs4/MapEncoder.java +++ b/container-search/src/main/java/com/yahoo/fs4/MapEncoder.java @@ -21,7 +21,9 @@ public class MapEncoder { // TODO: Time to refactor private static byte [] getUtf8(Object value) { - if (value instanceof Tensor) { + if (value == null) { + return Utf8.toBytes(""); + } else if (value instanceof Tensor) { return TypedBinaryFormat.encode((Tensor)value); } else { return Utf8.toBytes(value.toString()); @@ -70,11 +72,7 @@ public class MapEncoder { buffer.putInt(utf8.length); buffer.put(utf8); Object value = entry.getValue(); - if (value == null) { - utf8 = Utf8.toBytes(""); - } else { - utf8 = getUtf8(value); - } + utf8 = getUtf8(value); buffer.putInt(utf8.length); buffer.put(utf8); } |