lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adrien Grand (JIRA)" <>
Subject [jira] [Commented] (LUCENE-7749) IndexOrDocValuesQuery not working with LRUQueryCache (?)
Date Mon, 03 Apr 2017 09:13:41 GMT


Adrien Grand commented on LUCENE-7749:

I agree it is a bit error-prone and removing the {{scorer}} method or hiding it would fix
the issue. The way I have been approaching this issue is that it is a fairly new and experimental
API so I'd like to keep it optional for now (ie. query impls do not have to implement this
method). Now that this feature is released, hopefully we'll get feedback about how well it
works, which will in-turn give us opportunities to improve this API and once we are more confident
about it, we can think about removing the trap that you observed. For the record, this is
my personal opinion and might not be shared by other people who work on this project.

> IndexOrDocValuesQuery not working with LRUQueryCache (?)
> --------------------------------------------------------
>                 Key: LUCENE-7749
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Martin Amirault
>            Priority: Minor
>         Attachments: LUCENE-7749.patch
> I was experimenting with 6.5.0-SNAPSHOT and could not see any performance improvement
using the new IndexOrDocValuesQuery where I would have expected some.
> I am using a basic FILTER query (term + point/dv range), along with IndexSearcher#search.

> Looking at the stack trace it seems that LRUQueryCache#CachingWrapperWeight not delegating
the scorerSupplier method is the reason. 
> Maybe it is on purpose for the result to be cacheable ? Does that mean IndexOrDocValuesQuery
is not useable with the default IndexSearcher cache ? (Or maybe am I just completely misusing
the  IndexOrDocValuesQuery feature ?)
> Here is a thread dump of the call to IndexOrDocValuesQuery#scorerSupplier
> {quote}
> 	  at$1.scorerSupplier(
> 	  at$1.scorer(
> 	  at$CachingWrapperWeight.scorer(
> 	  at
> 	  at
> 	  at
> 	  at
> 	  at
> 	  at$CachingWrapperWeight.cache(
> 	  at$CachingWrapperWeight.bulkScorer(
> 	  at
> 	  at
> {quote}

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message