You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Deepika Sharma (Jira)" <ji...@apache.org> on 2022/05/18 17:35:00 UTC
[jira] [Comment Edited] (LUCENE-10544) Should ExitableTermsEnum wrap postings and impacts?
[ https://issues.apache.org/jira/browse/LUCENE-10544?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17529965#comment-17529965 ]
Deepika Sharma edited comment on LUCENE-10544 at 5/18/22 5:34 PM:
------------------------------------------------------------------
Yeah, I think you’re right [~jpountz] about the BulkScorer#score. One edge case though would probably be if a user passes their own BulkScorer, in which case this approach might not work properly. I guess what we could do is to allow a user to use a custom BulkScorer, when timeout is enabled, but this might not be a desirable restriction.
was (Author: JIRAUSER288832):
Yeah, I think you’re right [~jpountz] about the BulkScorer#score. One edge case though would probably be if a user passes their own BulkScorer, in which case this approach might not work properly. I guess what we could do is to allow a user to use a custom BulkScorer, when timeout is enabled, but this might not be a desirable restriction.
> Should ExitableTermsEnum wrap postings and impacts?
> ---------------------------------------------------
>
> Key: LUCENE-10544
> URL: https://issues.apache.org/jira/browse/LUCENE-10544
> Project: Lucene - Core
> Issue Type: Bug
> Components: core/index
> Reporter: Greg Miller
> Priority: Major
>
> While looking into options for LUCENE-10151, I noticed that {{ExitableDirectoryReader}} doesn't actually do any timeout checking once you start iterating postings/impacts. It *does* create a {{ExitableTermsEnum}} wrapper when loading a {{{}TermsEnum{}}}, but that wrapper doesn't do anything to wrap postings or impacts. So timeouts will be enforced when moving to the "next" term, but not when iterating the postings/impacts associated with a term.
> I think we ought to wrap the postings/impacts as well with some form of timeout checking so timeouts can be enforced on long-running queries. I'm not sure why this wasn't done originally (back in 2014), but it was questioned back in 2020 on the original Jira SOLR-5986. Does anyone know of a good reason why we shouldn't enforce timeouts in this way?
> Related, we may also want to wrap things like {{seekExact}} and {{seekCeil}} given that only {{next}} is being wrapped currently.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org