Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-2860

SegmentInfo.sizeInBytes ignore includeDocStore when caching

Log workAgile BoardRank to TopRank to BottomArchiveAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete CommentsDeleteAdd fieldWhere is my field?Permission helperNotification helper
    XMLWordPrintableJSON

Details

    • New, Patch Available

    Description

      I noticed that SegmentInfo's sizeInBytes cache is potentially buggy – it doesn't take into account 'includeDocStores'. I.e., if you call it once w/ 'false' (sizeInBytes won't include the store files) and then with 'true' (or vice versa), you won't get the right sizeInBytes (it won't re-compute, with the store files).

      I'll fix and add a test case demonstrating the bug.

      Attachments

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            shaie Shai Erera Assign to me
            shaie Shai Erera
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment