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

Shai Erera commented on LUCENE-5801:
------------------------------------

Actually, this isn't the correct fix. I modified the test to index two 
dimensions, one under custom indexFieldName {{$tags}} and one under the 
default. So facetsConfig.getDimConfigs() returns only the {{$tags}} facets. If 
I add the DEFAULT_INDEX_FIELD_NAME (always) then it solves the problem, but 
then if the app never uses that field name for faceting, but adds it as another 
BDV, we try to map ordinals ... I think it's not very likely that this happens 
though, so I'm willing to go with that fix.

I'm working on a patch with improved test.

> Resurrect org.apache.lucene.facet.util.OrdinalMappingAtomicReader
> -----------------------------------------------------------------
>
>                 Key: LUCENE-5801
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5801
>             Project: Lucene - Core
>          Issue Type: Bug
>    Affects Versions: 4.7
>            Reporter: Nicola Buso
>            Assignee: Shai Erera
>             Fix For: 5.0, 4.10
>
>         Attachments: LUCENE-5801.patch, LUCENE-5801.patch, LUCENE-5801.patch, 
> LUCENE-5801_1.patch, LUCENE-5801_2.patch
>
>
> from lucene > 4.6.1 the class:
> org.apache.lucene.facet.util.OrdinalMappingAtomicReader
> was removed; resurrect it because used merging indexes related to merged 
> taxonomies.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

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

Reply via email to