[kudu] branch master updated: [fs] Fix duplicated sync created_dirs in FsManager::Open()

[adar]

This is an automated email from the ASF dual-hosted git repository.

adar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 2af4132  [fs] Fix duplicated sync created_dirs in FsManager::Open()
2af4132 is described below

commit 2af4132ebf2dc67c3c1299aa80c8abd5c63fd778
Author: lingbin <lingbi...@gmail.com>
AuthorDate: Mon Oct 28 18:55:40 2019 +0800

    [fs] Fix duplicated sync created_dirs in FsManager::Open()
    
    Change-Id: If3c91bfdce44f4b7d7696542d4a7dfb7a3e59654
    Reviewed-on: http://gerrit.cloudera.org:8080/14558
    Tested-by: Kudu Jenkins
    Reviewed-by: Adar Dembo <a...@cloudera.com>
---
 src/kudu/fs/fs_manager.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/kudu/fs/fs_manager.cc b/src/kudu/fs/fs_manager.cc
index 605fd72..0979f32 100644
--- a/src/kudu/fs/fs_manager.cc
+++ b/src/kudu/fs/fs_manager.cc
@@ -435,7 +435,7 @@ Status FsManager::Open(FsReport* report) {
   if (FLAGS_enable_data_block_fsync) {
     // Files/directories created by the directory manager in the fs roots have
     // been synchronized, so now is a good time to sync the roots themselves.
-    WARN_NOT_OK(env_util::SyncAllParentDirs(env_, created_dirs, created_dirs),
+    WARN_NOT_OK(env_util::SyncAllParentDirs(env_, created_dirs, created_files),
                 "could not sync newly created fs roots");
   }