[PATCH] lib/core/coio_file: Use eio_sendfile_sync instead of a chunk mode

Cyrill Gorcunov gorcunov at gmail.com
Fri Apr 12 17:59:25 MSK 2019


On Fri, Apr 12, 2019 at 05:53:47PM +0300, Vladimir Davydov wrote:
> 
> True, I don't think it's worth testing it.
> 
> I just wanted to say that we need to make sure it works - check it
> manually and I will too.

Current tests are passed even without "cycled sendfile" simply
because current tests are not operating with huge files (i ran
them manually before sending the patch). I'll recheck everything
once again before sending v2. Thanks for review!

> Alternatively, we could add an error injection, reducing the sendfile
> copy size, but I think it's not really necessary.

OK



More information about the Tarantool-patches mailing list