benwtrent commented on PR #13202: URL: https://github.com/apache/lucene/pull/13202#issuecomment-2024967034
> unit vectors, 100 dimensions, indexed first 1M vectors, @kaivalnp AH, ok, only 100 dimensions, this explains the overly large impact that checking for timeout had, as there is very little work being done on distance calculations. I would expect the percentage of compute to drop as the amount of work done by vector comparisons to increase. I have a dumb question about timeouts. I am guessing the `QueryTimeout` object has to do some global syncing to determine the current runtime to check. Is that done on every single vector operation? If so, could we make sure its only done every `N` operations? Timeouts are already approximate, I understand the need for having a more strict timeout, but I am not sure we should check for timeout on every single vector. -- 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...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
