update destructors

db/column_family.h

@@ -168,7 +168,7 @@ class ColumnFamilyHandleImpl : public ColumnFamilyHandle {
   ColumnFamilyHandleImpl(ColumnFamilyData* cfd, DBImpl* db,
                          InstrumentedMutex* mutex);
   // destroy without mutex
-  virtual ~ColumnFamilyHandleImpl();
+  virtual ~ColumnFamilyHandleImpl() override;
   virtual ColumnFamilyData* cfd() const { return cfd_; }
   virtual DBImpl* db() const { return db_; }
 

db/compaction/compaction_picker.h

@@ -255,7 +255,7 @@ class NullCompactionPicker : public CompactionPicker {
   NullCompactionPicker(const ImmutableOptions& ioptions,
                        const InternalKeyComparator* icmp)
       : CompactionPicker(ioptions, icmp) {}
-  virtual ~NullCompactionPicker() {}
+  virtual ~NullCompactionPicker() override {}
 
   // Always return "nullptr"
   Compaction* PickCompaction(

db/db_impl/db_impl.h

@@ -195,7 +195,7 @@ class DBImpl : public DB {
   DBImpl(const DBImpl&) = delete;
   void operator=(const DBImpl&) = delete;
 
-  virtual ~DBImpl();
+  virtual ~DBImpl() override;
 
   // ---- Implementations of the DB interface ----
 
@@ -1921,8 +1921,8 @@ class DBImpl : public DB {
     const InternalKey* begin = nullptr;  // nullptr means beginning of key range
     const InternalKey* end = nullptr;    // nullptr means end of key range
     InternalKey* manual_end = nullptr;   // how far we are compacting
-    InternalKey tmp_storage;      // Used to keep track of compaction progress
-    InternalKey tmp_storage1;     // Used to keep track of compaction progress
+    InternalKey tmp_storage;   // Used to keep track of compaction progress
+    InternalKey tmp_storage1;  // Used to keep track of compaction progress
 
     // When the user provides a canceled pointer in CompactRangeOptions, the
     // above varaibe is the reference of the user-provided

db/db_impl/db_impl_follower.h

@@ -19,7 +19,7 @@ class DBImplFollower : public DBImplSecondary {
  public:
   DBImplFollower(const DBOptions& db_options, std::unique_ptr<Env>&& env,
                  const std::string& dbname, std::string src_path);
-  ~DBImplFollower();
+  ~DBImplFollower() override;
 
   Status Close() override;
 

db/db_impl/db_impl_readonly.h

@@ -5,7 +5,6 @@
 
 #pragma once
 
-
 #include <string>
 #include <vector>
 
@@ -21,7 +20,7 @@ class DBImplReadOnly : public DBImpl {
   DBImplReadOnly(const DBImplReadOnly&) = delete;
   void operator=(const DBImplReadOnly&) = delete;
 
-  virtual ~DBImplReadOnly();
+  virtual ~DBImplReadOnly() override;
 
   // Implementations of the DB interface
   using DBImpl::GetImpl;

db/dbformat.h

@@ -396,7 +396,7 @@ class InternalKeyComparator
   //    overhead, set `named` to false. In that case, `Name()` will return a
   //    generic name that is non-specific to the underlying comparator.
   explicit InternalKeyComparator(const Comparator* c) : user_comparator_(c) {}
-  virtual ~InternalKeyComparator() {}
+  virtual ~InternalKeyComparator() override {}
 
   int Compare(const Slice& a, const Slice& b) const override;
 

db/experimental.cc

@@ -37,7 +37,6 @@ Status PromoteL0(DB* db, ColumnFamilyHandle* column_family, int target_level) {
   return db->PromoteL0(column_family, target_level);
 }
 
-
 Status SuggestCompactRange(DB* db, const Slice* begin, const Slice* end) {
   return SuggestCompactRange(db, db->DefaultColumnFamily(), begin, end);
 }
@@ -517,7 +516,7 @@ class SstQueryFilterConfigImpl : public SstQueryFilterConfig {
       const KeySegmentsExtractor::KeyCategorySet& categories)
       : input_(input), categories_(categories) {}
 
-  virtual ~SstQueryFilterConfigImpl() = default;
+  virtual ~SstQueryFilterConfigImpl() override = default;
 
   virtual std::unique_ptr<SstQueryFilterBuilder> NewBuilder(
       bool sanity_checks) const = 0;

db/forward_iterator.h

@@ -4,13 +4,12 @@
 //  (found in the LICENSE.Apache file in the root directory).
 #pragma once
 
-#include "rocksdb/comparator.h"
-
 #include <queue>
 #include <string>
 #include <vector>
 
 #include "memory/arena.h"
+#include "rocksdb/comparator.h"
 #include "rocksdb/db.h"
 #include "rocksdb/iterator.h"
 #include "rocksdb/options.h"
@@ -55,7 +54,7 @@ class ForwardIterator : public InternalIterator {
   ForwardIterator(DBImpl* db, const ReadOptions& read_options,
                   ColumnFamilyData* cfd, SuperVersion* current_sv = nullptr,
                   bool allow_unprepared_value = false);
-  virtual ~ForwardIterator();
+  virtual ~ForwardIterator() override;
 
   void SeekForPrev(const Slice& /*target*/) override {
     status_ = Status::NotSupported("ForwardIterator::SeekForPrev()");

db/snapshot_checker.h

@@ -26,7 +26,7 @@ class SnapshotChecker {
 
 class DisableGCSnapshotChecker : public SnapshotChecker {
  public:
-  virtual ~DisableGCSnapshotChecker() {}
+  virtual ~DisableGCSnapshotChecker() override {}
   SnapshotCheckerResult CheckInSnapshot(
       SequenceNumber /*sequence*/,
       SequenceNumber /*snapshot_sequence*/) const override {
@@ -46,7 +46,7 @@ class WritePreparedTxnDB;
 class WritePreparedSnapshotChecker : public SnapshotChecker {
  public:
   explicit WritePreparedSnapshotChecker(WritePreparedTxnDB* txn_db);
-  virtual ~WritePreparedSnapshotChecker() {}
+  virtual ~WritePreparedSnapshotChecker() override {}
 
   SnapshotCheckerResult CheckInSnapshot(
       SequenceNumber sequence, SequenceNumber snapshot_sequence) const override;

db/table_properties_collector.h

@@ -66,7 +66,7 @@ class UserKeyTablePropertiesCollector : public InternalTblPropColl {
   explicit UserKeyTablePropertiesCollector(TablePropertiesCollector* collector)
       : collector_(collector) {}
 
-  virtual ~UserKeyTablePropertiesCollector() {}
+  virtual ~UserKeyTablePropertiesCollector() override {}
 
   Status InternalAdd(const Slice& key, const Slice& value,
                      uint64_t file_size) override;

db/write_controller.h

@@ -128,21 +128,21 @@ class StopWriteToken : public WriteControllerToken {
  public:
   explicit StopWriteToken(WriteController* controller)
       : WriteControllerToken(controller) {}
-  virtual ~StopWriteToken();
+  virtual ~StopWriteToken() override;
 };
 
 class DelayWriteToken : public WriteControllerToken {
  public:
   explicit DelayWriteToken(WriteController* controller)
       : WriteControllerToken(controller) {}
-  virtual ~DelayWriteToken();
+  virtual ~DelayWriteToken() override;
 };
 
 class CompactionPressureToken : public WriteControllerToken {
  public:
   explicit CompactionPressureToken(WriteController* controller)
       : WriteControllerToken(controller) {}
-  virtual ~CompactionPressureToken();
+  virtual ~CompactionPressureToken() override;
 };
 
 }  // namespace ROCKSDB_NAMESPACE

env/env_encryption_ctr.h

@@ -5,7 +5,6 @@
 
 #pragma once
 
-
 #include "rocksdb/env_encryption.h"
 
 namespace ROCKSDB_NAMESPACE {
@@ -24,8 +23,8 @@ class CTRCipherStream final : public BlockAccessCipherStream {
  public:
   CTRCipherStream(const std::shared_ptr<BlockCipher>& c, const char* iv,
                   uint64_t initialCounter)
-      : cipher_(c), iv_(iv, c->BlockSize()), initialCounter_(initialCounter){}
-  virtual ~CTRCipherStream(){}
+      : cipher_(c), iv_(iv, c->BlockSize()), initialCounter_(initialCounter) {}
+  virtual ~CTRCipherStream() override {}
 
   size_t BlockSize() override { return cipher_->BlockSize(); }
 
@@ -55,7 +54,7 @@ class CTREncryptionProvider : public EncryptionProvider {
  public:
   explicit CTREncryptionProvider(
       const std::shared_ptr<BlockCipher>& c = nullptr);
-  virtual ~CTREncryptionProvider() {}
+  virtual ~CTREncryptionProvider() override {}
 
   static const char* kClassName() { return "CTR"; }
   const char* Name() const override { return kClassName(); }
@@ -94,4 +93,3 @@ Status NewEncryptedFileSystemImpl(
     std::unique_ptr<FileSystem>* fs);
 
 }  // namespace ROCKSDB_NAMESPACE
-

env/fs_on_demand.h

@@ -100,7 +100,7 @@ class OnDemandSequentialFile : public FSSequentialFile {
         eof_(false),
         offset_(0) {}
 
-  virtual ~OnDemandSequentialFile() {}
+  virtual ~OnDemandSequentialFile() override {}
 
   IOStatus Read(size_t n, const IOOptions& options, Slice* result,
                 char* scratch, IODebugContext* dbg) override;

env/io_posix.h

@@ -273,7 +273,7 @@ class PosixSequentialFile : public FSSequentialFile {
  public:
   PosixSequentialFile(const std::string& fname, FILE* file, int fd,
                       size_t logical_block_size, const EnvOptions& options);
-  virtual ~PosixSequentialFile();
+  virtual ~PosixSequentialFile() override;
 
   IOStatus Read(size_t n, const IOOptions& opts, Slice* result, char* scratch,
                 IODebugContext* dbg) override;
@@ -326,7 +326,7 @@ class PosixRandomAccessFile : public FSRandomAccessFile {
                         ThreadLocalPtr* thread_local_io_urings
 #endif
   );
-  virtual ~PosixRandomAccessFile();
+  virtual ~PosixRandomAccessFile() override;
 
   IOStatus Read(uint64_t offset, size_t n, const IOOptions& opts, Slice* result,
                 char* scratch, IODebugContext* dbg) const override;
@@ -375,7 +375,7 @@ class PosixWritableFile : public FSWritableFile {
   explicit PosixWritableFile(const std::string& fname, int fd,
                              size_t logical_block_size,
                              const EnvOptions& options);
-  virtual ~PosixWritableFile();
+  virtual ~PosixWritableFile() override;
 
   // Need to implement this so the file is truncated correctly
   // with direct I/O
@@ -431,7 +431,7 @@ class PosixMmapReadableFile : public FSRandomAccessFile {
  public:
   PosixMmapReadableFile(const int fd, const std::string& fname, void* base,
                         size_t length, const EnvOptions& options);
-  virtual ~PosixMmapReadableFile();
+  virtual ~PosixMmapReadableFile() override;
   IOStatus Read(uint64_t offset, size_t n, const IOOptions& opts, Slice* result,
                 char* scratch, IODebugContext* dbg) const override;
   void Hint(AccessPattern pattern) override;
@@ -470,7 +470,7 @@ class PosixMmapFile : public FSWritableFile {
  public:
   PosixMmapFile(const std::string& fname, int fd, size_t page_size,
                 const EnvOptions& options);
-  ~PosixMmapFile();
+  ~PosixMmapFile() override;
 
   // Means Close() will properly take care of truncate
   // and it does not need any additional information
@@ -501,7 +501,7 @@ class PosixRandomRWFile : public FSRandomRWFile {
  public:
   explicit PosixRandomRWFile(const std::string& fname, int fd,
                              const EnvOptions& options);
-  virtual ~PosixRandomRWFile();
+  virtual ~PosixRandomRWFile() override;
 
   IOStatus Write(uint64_t offset, const Slice& data, const IOOptions& opts,
                  IODebugContext* dbg) override;
@@ -522,13 +522,13 @@ class PosixRandomRWFile : public FSRandomRWFile {
 struct PosixMemoryMappedFileBuffer : public MemoryMappedFileBuffer {
   PosixMemoryMappedFileBuffer(void* _base, size_t _length)
       : MemoryMappedFileBuffer(_base, _length) {}
-  virtual ~PosixMemoryMappedFileBuffer();
+  virtual ~PosixMemoryMappedFileBuffer() override;
 };
 
 class PosixDirectory : public FSDirectory {
  public:
   explicit PosixDirectory(int fd, const std::string& directory_name);
-  ~PosixDirectory();
+  ~PosixDirectory() override;
   IOStatus Fsync(const IOOptions& opts, IODebugContext* dbg) override;
 
   IOStatus Close(const IOOptions& opts, IODebugContext* dbg) override;

file/sst_file_manager_impl.h

@@ -34,7 +34,7 @@ class SstFileManagerImpl : public SstFileManager {
   SstFileManagerImpl(const SstFileManagerImpl& sfm) = delete;
   SstFileManagerImpl& operator=(const SstFileManagerImpl& sfm) = delete;
 
-  ~SstFileManagerImpl();
+  ~SstFileManagerImpl() override;
 
   // DB will call OnAddFile whenever a new sst/blob file is added.
   Status OnAddFile(const std::string& file_path);

include/rocksdb/compaction_filter.h

@@ -177,7 +177,7 @@ class CompactionFilter : public Customizable {
     static const int kUnknownStartLevel = -1;
   };
 
-  virtual ~CompactionFilter() {}
+  virtual ~CompactionFilter() override {}
   static const char* Type() { return "CompactionFilter"; }
   static Status CreateFromString(const ConfigOptions& config_options,
                                  const std::string& name,
@@ -348,7 +348,7 @@ class CompactionFilter : public Customizable {
 // including data loss, unreported corruption, deadlocks, and more.
 class CompactionFilterFactory : public Customizable {
  public:
-  virtual ~CompactionFilterFactory() {}
+  virtual ~CompactionFilterFactory() override {}
   static const char* Type() { return "CompactionFilterFactory"; }
   static Status CreateFromString(
       const ConfigOptions& config_options, const std::string& name,

include/rocksdb/env_encryption.h

@@ -5,7 +5,6 @@
 
 #pragma once
 
-
 #include <string>
 
 #include "rocksdb/customizable.h"
@@ -32,7 +31,7 @@ std::shared_ptr<FileSystem> NewEncryptedFS(
 // blocks). E.g. CTR (Counter operation mode) supports this requirement.
 class BlockAccessCipherStream {
  public:
-  virtual ~BlockAccessCipherStream(){}
+  virtual ~BlockAccessCipherStream() {}
 
   // BlockSize returns the size of each block supported by this cipher stream.
   virtual size_t BlockSize() = 0;
@@ -67,7 +66,7 @@ class BlockAccessCipherStream {
 // including data loss, unreported corruption, deadlocks, and more.
 class BlockCipher : public Customizable {
  public:
-  virtual ~BlockCipher() {}
+  virtual ~BlockCipher() override {}
 
   // Creates a new BlockCipher from the input config_options and value
   // The value describes the type of provider (and potentially optional
@@ -114,7 +113,7 @@ class BlockCipher : public Customizable {
 // including data loss, unreported corruption, deadlocks, and more.
 class EncryptionProvider : public Customizable {
  public:
-  virtual ~EncryptionProvider() {}
+  virtual ~EncryptionProvider() override {}
 
   // Creates a new EncryptionProvider from the input config_options and value.
   // The value describes the type of provider (and potentially optional
@@ -360,4 +359,3 @@ class EncryptedFileSystem : public FileSystemWrapper {
   }
 };
 }  // namespace ROCKSDB_NAMESPACE
-

include/rocksdb/file_system.h

@@ -296,7 +296,7 @@ class FileSystem : public Customizable {
   // No copying allowed
   FileSystem(const FileSystem&) = delete;
 
-  virtual ~FileSystem();
+  virtual ~FileSystem() override;
 
   static const char* Type() { return "FileSystem"; }
   static const char* kDefaultName() { return "DefaultFileSystem"; }

include/rocksdb/filter_policy.h

@@ -88,7 +88,7 @@ struct FilterBuildingContext {
 // GetBuilderWithContext.
 class FilterPolicy : public Customizable {
  public:
-  virtual ~FilterPolicy();
+  virtual ~FilterPolicy() override;
   static const char* Type() { return "FilterPolicy"; }
 
   // The name used for identifying whether a filter on disk is readable

include/rocksdb/flush_block_policy.h

@@ -53,7 +53,7 @@ class FlushBlockPolicyFactory : public Customizable {
       const BlockBasedTableOptions& table_options,
       const BlockBuilder& data_block_builder) const = 0;
 
-  virtual ~FlushBlockPolicyFactory() {}
+  virtual ~FlushBlockPolicyFactory() override {}
 };
 
 class FlushBlockBySizePolicyFactory : public FlushBlockPolicyFactory {

include/rocksdb/merge_operator.h

@@ -52,7 +52,7 @@ class Logger;
 // including data loss, unreported corruption, deadlocks, and more.
 class MergeOperator : public Customizable {
  public:
-  virtual ~MergeOperator() {}
+  virtual ~MergeOperator() override {}
   static const char* Type() { return "MergeOperator"; }
   static Status CreateFromString(const ConfigOptions& opts,
                                  const std::string& id,

include/rocksdb/slice_transform.h

@@ -34,7 +34,7 @@ struct ConfigOptions;
 // including data loss, unreported corruption, deadlocks, and more.
 class SliceTransform : public Customizable {
  public:
-  virtual ~SliceTransform(){}
+  virtual ~SliceTransform() override {}
 
   // Return the name of this transformation.
   const char* Name() const override = 0;