[LUCENE-2860] SegmentInfo.sizeInBytes ignore includeDocStore when caching

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.1, 4.0-ALPHA
• Component/s: core/index
• Lucene Fields: New, Patch Available
• Assignee: Shai Erera
• Reporter: Shai Erera
• Created: 12/Jan/11 04:44
• Updated: 30/Mar/11 15:50
• Resolved: 12/Jan/11 14:30

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.