GitHub user s1monw opened a pull request:

    https://github.com/apache/lucene-solr/pull/516

    LUCENE-8594: DV update are broken for updates on new field

    A segmemnt written with Lucene70Codec failes if it ties to update
    a DV field that didn't exist in the index before it was upgraded to
    Lucene80Codec. We bake the DV format into the FieldInfo when it's used
    the first time and therefor never go to the codec if we need to update.
    yet on a field that didn't exist before and was added during an indexing
    operation we have to consult the coded and get an exception.
    This change fixes this issue and adds the relevant bwc tests.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/s1monw/lucene-solr fix_dv_update_on_old_index

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/lucene-solr/pull/516.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #516
    
----
commit 45bb0938105cae6aaaa62d04f757e1cfe70ecb76
Author: Simon Willnauer <simonw@...>
Date:   2018-12-06T10:31:02Z

    LUCENE-8594: DV update are broken for updates on new field
    
    A segmemnt written with Lucene70Codec failes if it ties to update
    a DV field that didn't exist in the index before it was upgraded to
    Lucene80Codec. We bake the DV format into the FieldInfo when it's used
    the first time and therefor never go to the codec if we need to update.
    yet on a field that didn't exist before and was added during an indexing
    operation we have to consult the coded and get an exception.
    This change fixes this issue and adds the relevant bwc tests.

----


---

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

Reply via email to