Cleanup: Remove obsolete TODO from SparseVectorQueryBuilder (elastic#134702)

* Removed the TODO

* Updated the sparevector similar to Semantic query builder

* Update docs/changelog/134702.yaml

* Remove obsolete TODO comment from SparseVectorQueryBuilder

* [CI] Auto commit changes from spotless

* Modified the TODO to make more sense

* Delete docs/changelog/134702.yaml

* Update x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/search/SparseVectorQueryBuilder.java

Co-authored-by: Kathleen DeRusso <[email protected]>

* Update SparseVectorQueryBuilder.java

---------

Co-authored-by: elasticsearchmachine <[email protected]>
Co-authored-by: Kathleen DeRusso <[email protected]>

Author: 3 people

x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/search/SparseVectorQueryBuilder.java

@@ -271,7 +271,7 @@ protected QueryBuilder doRewrite(QueryRewriteContext queryRewriteContext) {
             throw new IllegalArgumentException("inference_id required to perform vector search on query string");
         }
 
-        // TODO move this to xpack core and use inference APIs
+        // TODO: Move this class to `server` and update to use InferenceAction.Request
         CoordinatedInferenceAction.Request inferRequest = CoordinatedInferenceAction.Request.forTextInput(
             inferenceId,
             List.of(query),