lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (JIRA)" <>
Subject [jira] [Commented] (SOLR-1387) Add more search options for filtering field facets.
Date Fri, 27 Mar 2015 16:12:55 GMT


Michael McCandless commented on SOLR-1387:

I'm concerned about the StringHelper.contains that was added for this issue:

  * Its signature implies it operates on BytesRef, but under the hood it secretly assumes
the bytes are valid UTF-8 (only for the ignoreCase=true case)

  * It also secretly assumes Locale.ENGLISH for downcasing but the incoming UTF-8 bytes may
not be English

  * It has potentially poor performance compared to known algos e.g.

Can we move this method out for now, e.g. not put it in the shared StringHelper utility class?

> Add more search options for filtering field facets.
> ---------------------------------------------------
>                 Key: SOLR-1387
>                 URL:
>             Project: Solr
>          Issue Type: New Feature
>          Components: search
>            Reporter: Anil Khadka
>            Assignee: Alan Woodward
>             Fix For: Trunk, 5.1
>         Attachments: SOLR-1387.patch
> Currently for filtering the facets, we have to use prefix (which use String.startsWith()
in java). 
> We can add some parameters like
> * facet.iPrefix : this would act like case-insensitive search. (or --->  facet.prefix=a&facet.caseinsense=on)
> * facet.regex : this is pure regular expression search (which obviously would be expensive
if issued).
> Moreover, allowing multiple filtering for same field would be great like
> facet.prefix=a OR facet.prefix=A ... sth like this.
> All above concepts could be equally applicable to TermsComponent.

This message was sent by Atlassian JIRA

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

View raw message