apply review comment

Author: jimczi

docs/reference/elasticsearch/mapping-reference/semantic-text.md

@@ -411,6 +411,31 @@ PUT my-index-000004
 }
 ```
 
+To retrieve all fragments from the `semantic` highlighter in their original indexing order
+without scoring, use a `match_all` query as the `highlight_query`.
+This ensures fragments are returned in the order they appear in the document:
+
+```console
+POST test-index/_search
+{
+  "query": {
+    "ids": {
+      "values": ["1"]
+    }
+  },
+  "highlight": {
+    "fields": {
+      "my_semantic_field": {
+        "number_of_fragments": 5,        <1>
+        "highlight_query": { "match_all": {} }
+      }
+    }
+  }
+}
+```
+
+1. Returns the first 5 fragments. Increase this value to retrieve additional fragments.
+
 ### Customizing using ingest pipelines [custom-by-pipelines]
 ```{applies_to}
 stack: ga 9.0

x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/mapper/SemanticTextFieldMapper.java

@@ -1082,7 +1082,7 @@ public List<Object> fetchValues(Source source, int doc, List<Object> ignoredValu
                     it.advance(previousParent);
                 }
 
-                return onlyTextChunks ? fetchTextChunks(source, doc, it) : fetchFullChunks(source, doc, it);
+                return onlyTextChunks ? fetchTextChunks(source, doc, it) : fetchFullField(source, doc, it);
             }
 
             private List<Object> fetchTextChunks(Source source, int doc, DocIdSetIterator it) throws IOException {
@@ -1102,7 +1102,7 @@ private List<Object> fetchTextChunks(Source source, int doc, DocIdSetIterator it
                 return chunks;
             }
 
-            private List<Object> fetchFullChunks(Source source, int doc, DocIdSetIterator it) throws IOException {
+            private List<Object> fetchFullField(Source source, int doc, DocIdSetIterator it) throws IOException {
                 Map<String, List<SemanticTextField.Chunk>> chunkMap = new LinkedHashMap<>();
 
                 iterateChildDocs(doc, it, offset -> {