Add comments/permit unchecked error to close_db_dir pull requests (#10093)

Summary:
In [close_db_dir](https://github.com/facebook/rocksdb/pull/10049) pull request, some merging conflicts occurred (some comments and one line `s.PermitUncheckedError()` are missing). This pull request aims to put them back.

Pull Request resolved: https://github.com/facebook/rocksdb/pull/10093

Reviewed By: ajkr

Differential Revision: D36884117

Pulled By: littlepig2013

fbshipit-source-id: 8c0e2a8793fc52804067c511843bd1ff4912c1c3

diff --git a/db/column_family.cc b/db/column_family.cc
index e5ec0a6ef9931b6dd1e8f8a7fc0249f88cf5d24e..bac4c21a6652cfc382261b2d3ca3aeccd0ae5dc4 100644 (file)
--- a/db/column_family.cc
+++ b/db/column_family.cc
@@ -682,8 +682,10 @@ ColumnFamilyData::~ColumnFamilyData() {
       if (data_dir_ptr) {
         s = data_dir_ptr->Close(IOOptions(), nullptr);
         if (!s.ok()) {
+          // TODO(zichen): add `Status Close()` and `CloseDirectories()
           ROCKS_LOG_WARN(ioptions_.logger, "Ignoring error %s",
                          s.ToString().c_str());
+          s.PermitUncheckedError();
         }
       }
     }

diff --git a/env/io_posix.cc b/env/io_posix.cc
index c0860c37624a759ab20b1135bc2222e34c7af041..ef4975de5f3cae7a491e2d3eb9d283d35b919018 100644 (file)
--- a/env/io_posix.cc
+++ b/env/io_posix.cc
@@ -1661,6 +1661,8 @@ IOStatus PosixDirectory::Fsync(const IOOptions& opts, IODebugContext* dbg) {
   return FsyncWithDirOptions(opts, dbg, DirFsyncOptions());
 }
 
+// Users who want the file entries synced in Directory project must call a
+// Fsync or FsyncWithDirOptions function before Close
 IOStatus PosixDirectory::Close(const IOOptions& /*opts*/,
                                IODebugContext* /*dbg*/) {
   IOStatus s = IOStatus::OK();