alessandrobenedetti commented on code in PR #872:
URL: https://github.com/apache/solr/pull/872#discussion_r879247719
##########
solr/core/src/java/org/apache/solr/index/SlowCompositeReaderWrapper.java:
##########
@@ -282,9 +282,9 @@ public VectorValues getVectorValues(String field) {
}
@Override
- public TopDocs searchNearestVectors(String field, float[] target, int k,
Bits acceptDocs)
- throws IOException {
- return null; // because not supported. Throw UOE?
+ public TopDocs searchNearestVectors(
+ String field, float[] target, int k, Bits acceptDocs, int visitedLimit)
throws IOException {
Review Comment:
ah ok, I was just suggesting to change it as I thought is was more correct
in English.
If changing it brings confusion, let's keep it as it is
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
