Thank you Alessandro. I've file the jira with more details and example:
https://issues.apache.org/jira/browse/SOLR-17615

On Wed, Jan 8, 2025 at 8:56 AM Alessandro Benedetti <a.benede...@sease.io>
wrote:

> The best I got is :
>
>
> https://solr.apache.org/guide/solr/latest/query-guide/dense-vector-search.html#implicit-pre-filtering
>
> This piece of work was added by Chris and it's quite complicated.
> If you feel there's a bug, please elaborate as much as possible, reproduce
> it and open a Jira ticket.
>
> We as a community will be able to help and possibly you will be able to
> make a contribution :)
>
> Cheers
> --------------------------
> *Alessandro Benedetti*
> Director @ Sease Ltd.
> *Apache Lucene/Solr Committer*
> *Apache Solr PMC Member*
>
> e-mail: a.benede...@sease.io
>
>
> *Sease* - Information Retrieval Applied
> Consulting | Training | Open Source
>
> Website: Sease.io <http://sease.io/>
> LinkedIn <https://linkedin.com/company/sease-ltd> | Twitter
> <https://twitter.com/seaseltd> | Youtube
> <https://www.youtube.com/channel/UCDx86ZKLYNpI3gzMercM7BQ> | Github
> <https://github.com/seaseltd>
>
>
> On Mon, 6 Jan 2025 at 20:16, Yue Yu <yue...@bloomreach.com> wrote:
>
> > Hi All,
> >
> > I see there is a fix for KnnQParser to add selected fqs as pre-filter:
> > https://issues.apache.org/jira/browse/SOLR-16858
> > Chris Hostetter was using "multiselect faceting" as an example to
> > illustrate the use case.
> >
> > However the "multiselect faceting" is still not working well with vector
> > search:
> >
> > In *org/apache/solr/request/SimpleFacets.java*
> >
> > protected DocSet computeDocSet(DocSet baseDocSet, List<String>
> > excludeTagList)
> >     throws SyntaxError, IOException {
> >   ...
> >
> >      // add the base query
> >      if (!excludeSet.containsKey(rb.getQuery())) {
> >         qlist.add(rb.getQuery());
> >      }
> >
> > ...
> > }
> >
> > Here the base query returned by *rb.getQuery() *still contains all the
> fqs
> > as pre-filter including the fq that the facet request tries to exclude.
> >
> > I'm wondering if this is fixed in a later version?
> >
> > Best,
> >
> > Yue
> >
>

Reply via email to