[
https://issues.apache.org/jira/browse/LUCENE-9455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17215715#comment-17215715
]
Zach Chen commented on LUCENE-9455:
-----------------------------------
Thanks [~broustant]! I've created a PR for this change. Could you please take a
look and let me know if it looks good?
I'm also wondering if I need to run any benchmark tests for this, as they seems
to be run for some performance improving changes?
> ExitableTermsEnum (in ExitableDirectoryReader) should sample next()
> -------------------------------------------------------------------
>
> Key: LUCENE-9455
> URL: https://issues.apache.org/jira/browse/LUCENE-9455
> Project: Lucene - Core
> Issue Type: Improvement
> Components: core/other
> Reporter: David Smiley
> Priority: Major
> Labels: newdev
> Time Spent: 10m
> Remaining Estimate: 0h
>
> ExitableTermsEnum calls "checkAndThrow" on *every* call to next(). This is
> too expensive; it should sample. I observed ElasticSearch uses the same
> approach; I think Lucene would benefit from this:
> https://github.com/elastic/elasticsearch/blob/4af4eb99e18fdaadac879b1223e986227dd2ee71/server/src/main/java/org/elasticsearch/search/internal/ExitableDirectoryReader.java#L151
> CC [~jimczi]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
