On 28 May 2015 at 18:45, <stef...@apache.org> wrote: > Author: stefan2 > Date: Thu May 28 15:45:55 2015 > New Revision: 1682265 > > URL: http://svn.apache.org/r1682265 > Log: > Correctly fsync() after renames in FSFS on Win32. We must flush the disk > buffers after the rename, otherwise the metadata may not be persistent. > Moreover, if the rename is degraded to a copy by Win32, we won't even have > the complete file contents on disk without a buffer flush. > Are you sure about this change? From my analysis that I posted two years ago [1] metadata changes are persistent fine on Windows, the only thing we need to make sure to flush data changes before move. Did you able to reproduce data corruption on VM?
This change should significantly degrade performance of commit operation, so it should be done only if we are really sure that this code required. [1] http://svn.haxx.se/dev/archive-2013-05/0245.shtml -- Ivan Zhakov
