Fix up virtual destructors

tiledb/sm/consolidator/consolidator.cc

@@ -127,8 +127,6 @@ Consolidator::Consolidator(JobParent& parent)
   consolidator_memory_tracker_->set_type(MemoryTrackerType::CONSOLIDATOR);
 }
 
-Consolidator::~Consolidator() = default;
-
 /* ****************************** */
 /*               API              */
 /* ****************************** */

tiledb/sm/consolidator/consolidator.h

@@ -85,6 +85,11 @@ class Consolidator : public JobBranch {
   explicit Consolidator(JobParent& parent);
 
  public:
+  /**
+   * Virtual destructor is override from `JobBranch`.
+   */
+  ~Consolidator() override = default;
+
   /* ********************************* */
   /*          FACTORY METHODS          */
   /* ********************************* */
@@ -111,13 +116,6 @@ class Consolidator : public JobBranch {
   static ConsolidationMode mode_from_config(
       const Config& config, const bool vacuum_mode = false);
 
-  /* ********************************* */
-  /*            DESTRUCTORS            */
-  /* ********************************* */
-
-  /** Destructor. */
-  virtual ~Consolidator();
-
   /* ********************************* */
   /*                API                */
   /* ********************************* */

tiledb/sm/query/dimension_label/array_dimension_label_queries.h

@@ -66,6 +66,11 @@ class ArrayDimensionLabelQueries : public JobBranch {
   /** Default constructor is not C.41 compliant. */
   ArrayDimensionLabelQueries() = delete;
 
+  /**
+   * Virtual destructor is override from `JobBranch`.
+   */
+  ~ArrayDimensionLabelQueries() override = default;
+
   /**
    * Constructor.
    *

tiledb/sm/storage_manager/context.h

@@ -124,7 +124,7 @@ class Context : public ContextBase, public JobRoot {
   explicit Context(const Config&);
 
   /** Destructor. */
-  ~Context() = default;
+  ~Context() override = default;
 
   /* ********************************* */
   /*                API                */

tiledb/sm/storage_manager/job.h

@@ -143,6 +143,8 @@ struct JobResourceMixin {
         , activity_(activity) {
     }
 
+    virtual ~ParentMixin() = default;
+
     /**
      * Accessor for the resources of this Parent
      *

tiledb/sm/storage_manager/test/unit_job.cc

@@ -248,6 +248,7 @@ class TestJobRoot : protected TestJobRootBase, public JobRoot {
   TestJobRoot()
       : TestJobRootBase(), JobRoot(Base::sm_) {
   }
+  ~TestJobRoot() override = default;
   ContextResources& resources() const override {
     return Base::resources_;
   }
@@ -264,6 +265,7 @@ class TestJobBranch : public JobBranch {
   TestJobBranch(JobParent& parent)
       : JobBranch(parent) {
   }
+  ~TestJobBranch() override = default;
   ContextResources& resources() const override {
     return parent().resources();
   }