[LUCENE-3727] fix assertions/checks that use File.length() to use getFilePointer()

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.6
• Fix Version/s: 3.6
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Robert Muir
• Created: 27/Jan/12 13:39
• Updated: 28/Aug/22 13:06
• Resolved: 30/Jan/12 01:09

Description

This came up on this thread "Getting RuntimeException: after flush: fdx size mismatch while Indexing"
(http://www.lucidimagination.com/search/document/a8db01a220f0a126)

In trunk, a side effect of the codec refactoring is that these assertions were pushed into codecs as finish() before close().
they check getFilePointer() instead in this computation, which checks that lucene did its part (instead of falsely tripping if directory metadata is stale).

I think we should fix these checks/asserts on 3.x too