Fix FetchSourceContext serialization for mixed-version clusters (#129172)

jimczi · web-flow · commit c2484d697eb3 · 2025-06-12T02:18:08.000+10:00
Ensure the new exclude_vectors parameter is correctly serialized and
deserialized when operating in a mixed-version cluster. Given that
version 8.19 is not compatible with 9.0, we can assume that all versions
on or after 9.1 will support this new parameter.
diff --git a/server/src/main/java/org/elasticsearch/search/fetch/subphase/FetchSourceContext.java b/server/src/main/java/org/elasticsearch/search/fetch/subphase/FetchSourceContext.java
@@ -97,7 +97,7 @@ public void writeTo(StreamOutput out) throws IOException {
     }
 
     private static boolean isVersionCompatibleWithExcludeVectors(TransportVersion version) {
-        return version.isPatchFrom(TransportVersions.SEARCH_SOURCE_EXCLUDE_VECTORS_PARAM_8_19);
+        return version.onOrAfter(TransportVersions.SEARCH_SOURCE_EXCLUDE_VECTORS_PARAM_8_19);
     }
 
     public boolean fetchSource() {

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ public void writeTo(StreamOutput out) throws IOException {`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`private static boolean isVersionCompatibleWithExcludeVectors(TransportVersion version) {`
`100`		`- return version.isPatchFrom(TransportVersions.SEARCH_SOURCE_EXCLUDE_VECTORS_PARAM_8_19);`
	`100`	`+ return version.onOrAfter(TransportVersions.SEARCH_SOURCE_EXCLUDE_VECTORS_PARAM_8_19);`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`public boolean fetchSource() {`