Implement SortOptionsVariant in FieldSort (#1323) (#1329)

* Implement SortOptionsVariant in FieldSort Signed-off-by: Kosuke Suzuki <[email protected]> * Add change log Signed-off-by: Kosuke Suzuki <[email protected]> --------- Signed-off-by: Kosuke Suzuki <[email protected]> (cherry picked from commit f9540f7)
opensearch-project · Nov 25, 2024 · 6d4534e · 6d4534e
1 parent 008c013
commit 6d4534e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -17,6 +17,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 ### Fixed
 - Fixed `IcuCollationDecomposition`'s variants to align with those supported by OpenSearch ([#]())
 - Fixed don't invoke the mapper's serialize method for the RangeQuery JsonData raw value [#1309](https://github.com/opensearch-project/opensearch-java/pull/1309)
+- Fixed an issue where `FieldSort` was not implementing `SortOptionsVariant` ([#1323](https://github.com/opensearch-project/opensearch-java/pull/1323))
 
 ### Security
 

diff --git a/java-client/src/main/java/org/opensearch/client/opensearch/_types/FieldSort.java b/java-client/src/main/java/org/opensearch/client/opensearch/_types/FieldSort.java
@@ -49,7 +49,7 @@
 // typedef: _types.FieldSort
 
 @JsonpDeserializable
-public class FieldSort implements PlainJsonSerializable {
+public class FieldSort implements SortOptionsVariant, PlainJsonSerializable {
     // Single key dictionary
     private final String field;
 
@@ -94,6 +94,14 @@ public static FieldSort of(Function<Builder, ObjectBuilder<FieldSort>> fn) {
         return fn.apply(new Builder()).build();
     }
 
+    /**
+     * SortOptions variant kind.
+     */
+    @Override
+    public SortOptions.Kind _sortOptionsKind() {
+        return SortOptions.Kind.Field;
+    }
+
     /**
      * Required - The target field
      */