[ 
https://issues.apache.org/jira/browse/LUCENE-2805?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Simon Willnauer updated LUCENE-2805:
------------------------------------

    Attachment: LUCENE-2805_3x.patch

here is a patch against 3.x. I needed to change an assert in the backwards 
tests so that is potentially a bw break and I am not 100% sure how we handle 
this right now. Is changing the bw test ok in this case? If its really a bw 
break I guess we should also list it in the BW section in CHANGES.txt

> SegmentInfos shouldn't blindly increment version on commit
> ----------------------------------------------------------
>
>                 Key: LUCENE-2805
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2805
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Index
>            Reporter: Michael McCandless
>            Assignee: Simon Willnauer
>             Fix For: 3.1, 4.0
>
>         Attachments: LUCENE-2805.patch, LUCENE-2805.patch, LUCENE-2805.patch, 
> LUCENE-2805_3x.patch
>
>
> SegmentInfos currently increments version on the assumption that there are 
> always changes.
> But, both DirReader and IW are more careful about tracking whether there are 
> changes.  DirReader has hasChanges and IW has changeCount.  I think these 
> classes should notify the SIS when there are in fact changes; this will fix 
> the case Simon hit on fixing LUCENE-2082 when the NRT reader thought there 
> were changes, but in fact there weren't because IW simply committed the exact 
> SIS it already had.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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

Reply via email to