No :). Fixed. Thanks for catching!

On Fri, Sep 20, 2013 at 7:50 AM, Robert Muir <[email protected]> wrote:

> On Fri, Sep 20, 2013 at 12:39 AM,  <[email protected]> wrote:
> > @@ -501,7 +504,7 @@ class ReadersAndLiveDocs { // TODO (DVU_
> >      if (hasFieldUpdates) {
> >        info.advanceDocValuesGen();
> >        // copy all the updates to mergingUpdates, so they can later be
> applied to the merged segment
> > -      if (isMerging) {
> > +      if (isMerging || true) {
> >          copyUpdatesToMerging();
> >        }
> >        numericUpdates.clear();
>
> Is this really intentional?
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>

Reply via email to