You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Ryan Ernst (JIRA)" <ji...@apache.org> on 2015/12/04 11:17:11 UTC

[jira] [Commented] (LUCENE-6918) LRUQueryCache.onDocIdSetEviction should not be called when nothing is evicted

    [ https://issues.apache.org/jira/browse/LUCENE-6918?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15041379#comment-15041379 ] 

Ryan Ernst commented on LUCENE-6918:
------------------------------------

+1, looks good.

For the changes entry, I think you mean to remove the "not" from "is not only called"?

> LRUQueryCache.onDocIdSetEviction should not be called when nothing is evicted
> -----------------------------------------------------------------------------
>
>                 Key: LUCENE-6918
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6918
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Adrien Grand
>            Assignee: Adrien Grand
>            Priority: Minor
>             Fix For: 5.5
>
>         Attachments: LUCENE-6918.patch
>
>
> This method is confusing because it states it will be called "when one or more DocIdSets are removed from this cache" but may actually be called with zero docidsets when evicting a per-segment cache that did not contain any entries.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org