[
https://issues.apache.org/jira/browse/LUCENE-9455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17218367#comment-17218367
]
Bruno Roustant commented on LUCENE-9455:
----------------------------------------
I propose to also sample the call to QueryTimeout.shouldExit() in
ExitableTermsEnum constructor with
(System.identityHashCode(this) & TIMEOUT_CHECK_SAMPLING) == 0
> 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: 50m
> 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
