Comment

Signed-off-by: Finn Carroll <carrofin@amazon.com>
opensearch-project · Aug 23, 2024 · 59d9765 · 59d9765
1 parent 1103fbb
commit 59d9765
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/server/src/main/java/org/opensearch/search/fetch/serde/FetchSearchResultsSerDe.java b/server/src/main/java/org/opensearch/search/fetch/serde/FetchSearchResultsSerDe.java
@@ -21,15 +21,10 @@
  */
 public class FetchSearchResultsSerDe implements SerDe.StreamSerializer<FetchSearchResult>, SerDe.StreamDeserializer<FetchSearchResult> {
     /**
-     * TODO NOTE: FetchSearchResult inheritance structure is as follows.
+     * NOTE: FetchSearchResultSerDe should mirror class inheritence strucutre.
      * TransportMessage -> TransportResponse -> SearchPhaseResult -> FetchSearchResult.
-     * Serialization of parent classes is currently a no-op.
-     * For completeness these parent classes should be mirrored here respectively with:
-     * TransportMessageSerDe, TransportResponseSerDe, SearchPhaseResultSerDe.
-     * However, currently only SearchHitsSerDe is needed for serialization.
      *
-     * This is implicitely enforced by FetchSearchResult as well on the serialization side.
-     * writeTo doesn't call a parent implementation...
+     * Serialization of parent classes is currently a no-op so leaving as is for now.
      */
     SearchHitsSerDe searchHitsSerDe;