Reconstruct behaviour on IO error

[Adam Tauno Williams]

On Mon, 2010-10-04 at 14:26 -0500, Patrick Goetz wrote: 
> On 10/04/2010 02:07 PM, Bron Gondwana wrote:
> > Keep going and remove entires for messages with
> > IO errors reading their file, or bail and make the administrator fix the
> > underlying permissions first?  I can see arguments for both - though remember
> > that once you've removed the index record you can never go back under
> > strict UID existence semantics, so if the admin fixes the files later then
> > reconstruct will actually give them new UIDs.
> Definitely bail. 

+1

> Now I think that error should at least include the file name

Absolutely, yes.  Nothing is more annoying than "File not found."