[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files
From: |
Azat Khuzhin |
Subject: |
bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files |
Date: |
Tue, 27 May 2014 00:56:11 +0400 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, May 26, 2014 at 01:44:14PM -0700, Paul Eggert wrote:
> Azat Khuzhin wrote:
> >The current patch only look while files is created, but this is not
> >enough `I agree with you, it must check write(2) and fallback to creating
> >when write(2) will fail with ENOSPC.
> >This is what you mean?
>
> Yes.
>
> >when we have ENOSPC it
> >means that we already won't create any more files there, and one file
> >with relatively small size is not a big deal.
>
> OK. The point is that 'sort' shouldn't lose the data (including the
> possibly-incomplete trailing line) that's already in the temporary file when
> a write to that file fails.
Thanks for the explanation, I see what you mean.
>
> Also, the code could treat EIO like ENOSPC, I suppose, to be more robust in
> the presence of bad temporary devices.
>
> But beware file systems that report ENOSPC and EIO in a delayed fashion,
> i.e., not immediately upon the failing write, but somewhat later, typically
> when closing the output file.
Yeah, that's a good catch, I will keep this in mind when I will start
working on this.
Thanks,
Azat.
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Azat Khuzhin, 2014/05/11
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Paul Eggert, 2014/05/11
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Pádraig Brady, 2014/05/11
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Azat Khuzhin, 2014/05/14
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Pádraig Brady, 2014/05/14
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Paul Eggert, 2014/05/14
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Azat Khuzhin, 2014/05/26
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Paul Eggert, 2014/05/26
- bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files,
Azat Khuzhin <=
bug#17470: [PATCH] sort: rotate on ENOSPC while creating tmp files, Azat Khuzhin, 2014/05/11