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

Adrien Grand commented on LUCENE-9450:
--------------------------------------

Thinking a bit more about this, the approach of looking at the index created 
version means users won't benefit from this change on their existing indices 
and requires a bit more work in the short term, but it's also a bit cleaner and 
will help us get done with this change earlier?

Scenario 1: use a different field name for binary doc values
 - Lucene 9: writes binary doc values under field name $full_path_binary$, 
reads either stored fields for $full_path$ (Lucene 8 compatibility) or binary 
doc values for $full_path_binary$ (Lucene 9 compatibility).
 - Lucene 10: writes binary doc values under field name $full_path$, reads 
either binary doc values for $full_path_binary$ (Lucene 9 compatibility) or 
binary doc values for $full_path$ (Lucene 10 compatibility).
 - Lucene 11: migration is complete, Lucene always use binary doc values with 
field name $full_path$.

Scenario 2: use index created version
 - Lucene 9: reads/writes binary doc values or stored fields depending on the 
index created version.
 - Lucene 10: migration is complete, Lucene always use binary doc values with 
field name $full_path$.


> Taxonomy index should use DocValues not StoredFields
> ----------------------------------------------------
>
>                 Key: LUCENE-9450
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9450
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/facet
>    Affects Versions: 8.5.2
>            Reporter: Gautam Worah
>            Priority: Minor
>              Labels: performance
>             Fix For: main (9.0)
>
>         Attachments: LUCENE-9450-localrun.py-v1, wip_taxonomy_patch
>
>          Time Spent: 3h 50m
>  Remaining Estimate: 0h
>
> The taxonomy index that maps binning labels to ordinals was created before 
> Lucene added BinaryDocValues.
> I've attached a WIP patch (does not pass tests currently)
> Issue suggested by [~mikemccand]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to