lingbin opened a new pull request #2884: Add `sync_dir` interface to Env
URL: https://github.com/apache/incubator-doris/pull/2884
 
 
   when we need to ensure that **a newly-created file** is fully
   synchronized back to disk, we should call `fsync()` on the parent
   directory—that is, the directory containing the newly-created file.
   That is to say, In this situation, we should call `fsync()` on
   both the newly-created file and its parent directory.
   
   Unfortunately, currently in Doris, in any scenario, directories
   are not fsynced.
   
   This patch adds `sync_dir()` interface first, laying the goundwork
   for future fixes.
   
   This patch also remove unneeded private method `dir_exists()`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to