You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2018/07/05 22:20:43 UTC
[2/2] mesos git commit: Added override specifiers to virtual methods
of several isolators.
Added override specifiers to virtual methods of several isolators.
This is a follow-up change for disk/xfs, network/port_mapping and
network/ports isolators to avoid problems with virtual method signatures
in the future.
Review: https://reviews.apache.org/r/67821/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/16455857
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/16455857
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/16455857
Branch: refs/heads/master
Commit: 16455857ee00e98147d8cb9fb6f31b22554dfe52
Parents: ea38d7e
Author: Ilya Pronin <ip...@twopensource.com>
Authored: Thu Jul 5 15:20:33 2018 -0700
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Thu Jul 5 15:20:33 2018 -0700
----------------------------------------------------------------------
.../mesos/isolators/network/port_mapping.hpp | 38 ++++++++++----------
.../mesos/isolators/network/ports.hpp | 26 +++++++-------
.../containerizer/mesos/isolators/xfs/disk.hpp | 28 +++++++--------
3 files changed, 46 insertions(+), 46 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/16455857/src/slave/containerizer/mesos/isolators/network/port_mapping.hpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/isolators/network/port_mapping.hpp b/src/slave/containerizer/mesos/isolators/network/port_mapping.hpp
index 8d46ba5..c14a8cc 100644
--- a/src/slave/containerizer/mesos/isolators/network/port_mapping.hpp
+++ b/src/slave/containerizer/mesos/isolators/network/port_mapping.hpp
@@ -149,32 +149,32 @@ class PortMappingIsolatorProcess : public MesosIsolatorProcess
public:
static Try<mesos::slave::Isolator*> create(const Flags& flags);
- virtual ~PortMappingIsolatorProcess() {}
+ ~PortMappingIsolatorProcess() override {}
- virtual process::Future<Nothing> recover(
+ process::Future<Nothing> recover(
const std::vector<mesos::slave::ContainerState>& states,
- const hashset<ContainerID>& orphans);
+ const hashset<ContainerID>& orphans) override;
- virtual process::Future<Option<mesos::slave::ContainerLaunchInfo>> prepare(
+ process::Future<Option<mesos::slave::ContainerLaunchInfo>> prepare(
const ContainerID& containerId,
- const mesos::slave::ContainerConfig& containerConfig);
+ const mesos::slave::ContainerConfig& containerConfig) override;
- virtual process::Future<Nothing> isolate(
+ process::Future<Nothing> isolate(
const ContainerID& containerId,
- pid_t pid);
+ pid_t pid) override;
- virtual process::Future<mesos::slave::ContainerLimitation> watch(
- const ContainerID& containerId);
+ process::Future<mesos::slave::ContainerLimitation> watch(
+ const ContainerID& containerId) override;
- virtual process::Future<Nothing> update(
+ process::Future<Nothing> update(
const ContainerID& containerId,
- const Resources& resources);
+ const Resources& resources) override;
- virtual process::Future<ResourceStatistics> usage(
- const ContainerID& containerId);
+ process::Future<ResourceStatistics> usage(
+ const ContainerID& containerId) override;
- virtual process::Future<Nothing> cleanup(
- const ContainerID& containerId);
+ process::Future<Nothing> cleanup(
+ const ContainerID& containerId) override;
private:
struct Info
@@ -368,8 +368,8 @@ public:
Flags flags;
protected:
- virtual int execute();
- virtual flags::FlagsBase* getFlags() { return &flags; }
+ int execute() override;
+ flags::FlagsBase* getFlags() override { return &flags; }
};
@@ -397,8 +397,8 @@ public:
Flags flags;
protected:
- virtual int execute();
- virtual flags::FlagsBase* getFlags() { return &flags; }
+ int execute() override;
+ flags::FlagsBase* getFlags() override { return &flags; }
};
} // namespace slave {
http://git-wip-us.apache.org/repos/asf/mesos/blob/16455857/src/slave/containerizer/mesos/isolators/network/ports.hpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/isolators/network/ports.hpp b/src/slave/containerizer/mesos/isolators/network/ports.hpp
index 7e7a757..6944d01 100644
--- a/src/slave/containerizer/mesos/isolators/network/ports.hpp
+++ b/src/slave/containerizer/mesos/isolators/network/ports.hpp
@@ -52,34 +52,34 @@ public:
static Try<mesos::slave::Isolator*> create(const Flags& flags);
- virtual ~NetworkPortsIsolatorProcess() {}
+ ~NetworkPortsIsolatorProcess() override {}
- virtual bool supportsNesting();
+ bool supportsNesting() override;
- virtual process::Future<Nothing> recover(
+ process::Future<Nothing> recover(
const std::vector<mesos::slave::ContainerState>& states,
- const hashset<ContainerID>& orphans);
+ const hashset<ContainerID>& orphans) override;
- virtual process::Future<Option<mesos::slave::ContainerLaunchInfo>> prepare(
+ process::Future<Option<mesos::slave::ContainerLaunchInfo>> prepare(
const ContainerID& containerId,
- const mesos::slave::ContainerConfig& containerConfig);
+ const mesos::slave::ContainerConfig& containerConfig) override;
- virtual process::Future<mesos::slave::ContainerLimitation> watch(
- const ContainerID& containerId);
+ process::Future<mesos::slave::ContainerLimitation> watch(
+ const ContainerID& containerId) override;
- virtual process::Future<Nothing> update(
+ process::Future<Nothing> update(
const ContainerID& containerId,
- const Resources& resources);
+ const Resources& resources) override;
- virtual process::Future<Nothing> cleanup(
- const ContainerID& containerId);
+ process::Future<Nothing> cleanup(
+ const ContainerID& containerId) override;
// Public only for testing.
process::Future<Nothing> check(
const hashmap<ContainerID, IntervalSet<uint16_t>>& listeners);
protected:
- virtual void initialize();
+ void initialize() override;
private:
NetworkPortsIsolatorProcess(
http://git-wip-us.apache.org/repos/asf/mesos/blob/16455857/src/slave/containerizer/mesos/isolators/xfs/disk.hpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/isolators/xfs/disk.hpp b/src/slave/containerizer/mesos/isolators/xfs/disk.hpp
index e64a779..0891f77 100644
--- a/src/slave/containerizer/mesos/isolators/xfs/disk.hpp
+++ b/src/slave/containerizer/mesos/isolators/xfs/disk.hpp
@@ -40,36 +40,36 @@ class XfsDiskIsolatorProcess : public MesosIsolatorProcess
public:
static Try<mesos::slave::Isolator*> create(const Flags& flags);
- virtual ~XfsDiskIsolatorProcess();
+ ~XfsDiskIsolatorProcess() override;
process::PID<XfsDiskIsolatorProcess> self() const
{
return process::PID<XfsDiskIsolatorProcess>(this);
}
- virtual process::Future<Nothing> recover(
+ process::Future<Nothing> recover(
const std::vector<mesos::slave::ContainerState>& states,
- const hashset<ContainerID>& orphans);
+ const hashset<ContainerID>& orphans) override;
- virtual process::Future<Option<mesos::slave::ContainerLaunchInfo>> prepare(
+ process::Future<Option<mesos::slave::ContainerLaunchInfo>> prepare(
const ContainerID& containerId,
- const mesos::slave::ContainerConfig& containerConfig);
+ const mesos::slave::ContainerConfig& containerConfig) override;
- virtual process::Future<mesos::slave::ContainerLimitation> watch(
- const ContainerID& containerId);
+ process::Future<mesos::slave::ContainerLimitation> watch(
+ const ContainerID& containerId) override;
- virtual process::Future<Nothing> update(
+ process::Future<Nothing> update(
const ContainerID& containerId,
- const Resources& resources);
+ const Resources& resources) override;
- virtual process::Future<ResourceStatistics> usage(
- const ContainerID& containerId);
+ process::Future<ResourceStatistics> usage(
+ const ContainerID& containerId) override;
- virtual process::Future<Nothing> cleanup(
- const ContainerID& containerId);
+ process::Future<Nothing> cleanup(
+ const ContainerID& containerId) override;
protected:
- virtual void initialize();
+ void initialize() override;
private:
XfsDiskIsolatorProcess(