From 0b760ab83a42aee24860bfd08bf5e366bce22457 Mon Sep 17 00:00:00 2001 From: Andrew Ross Date: Wed, 18 Sep 2024 06:17:56 -0700 Subject: [PATCH] Serialize MediaType as an integer to avoid string parsing Resolves #15979 Signed-off-by: Andrew Ross --- .../org/opensearch/common/xcontent/XContentType.java | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/libs/x-content/src/main/java/org/opensearch/common/xcontent/XContentType.java b/libs/x-content/src/main/java/org/opensearch/common/xcontent/XContentType.java index 4611ee58d001b..bf2c17846c6e6 100644 --- a/libs/x-content/src/main/java/org/opensearch/common/xcontent/XContentType.java +++ b/libs/x-content/src/main/java/org/opensearch/common/xcontent/XContentType.java @@ -239,7 +239,7 @@ public XContentBuilder contentBuilder(final OutputStream os) throws IOException } }; - private int index; + private final int index; XContentType(int index) { this.index = index; @@ -259,15 +259,6 @@ public String format() { return subtype(); } - @Override - public void writeTo(StreamOutput output) throws IOException { - if (output.getVersion().onOrAfter(Version.V_3_0_0)) { - output.writeVInt(uniqueId()); - } else { - output.writeString(this.mediaType()); - } - } - @Override public int uniqueId() { return this.ordinal();