uschindler commented on PR #12281: URL: https://github.com/apache/lucene/pull/12281#issuecomment-1585731684
> The check belongs in KnnVectorField, not here. Yes and no. In KnnVectorField it needs to scan through all vector components to check that it is not finite. The check here that isFinite is only on return value of calculation. If one component is inFinite the result of calculation is infinite. The code only scans through vector to give a good exception. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
