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

Shai Erera commented on LUCENE-5933:
------------------------------------

Spans is not plural, it's the name of the class Spans :). This one 
wraps/filters a Spans impl, not SpanQuery. Therefore the decision would be 
between FilterSpans and SpansWrapper. I have no strong feelings about either. I 
used Filter since we have that in FilterDirectory and FilterAtomicReader

> Add FilterSpans to allow easily wrapping a Spans impl
> -----------------------------------------------------
>
>                 Key: LUCENE-5933
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5933
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/search
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>         Attachments: LUCNE-5933.patch
>
>
> I found this useful while working with spans recently. It's simple and 
> straightforward. I'll add a patch shortly.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to