bruno-roustant commented on pull request #430: URL: https://github.com/apache/lucene/pull/430#issuecomment-970150412
I tried different variations (e.g. median of min/max instead of min/max of medians) but without more gain. I finally tried to fully remove the top-k shortcut, and it actually becomes faster. My previous analysis was not correct. Even for k very close to 'from' or 'last', the new max of medians is nearly as fast as top-k. So I've removed top-k in the last version, and it simplifies the code. -- 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
