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 2016/02/21 13:54:44 UTC
mesos git commit: Replaced EXIT(1) with EXIT(EXIT_FAILURE) in the
executor driver.
Repository: mesos
Updated Branches:
refs/heads/master 1e65c7fd0 -> de1434978
Replaced EXIT(1) with EXIT(EXIT_FAILURE) in the executor driver.
Review: https://reviews.apache.org/r/43761/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/de143497
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/de143497
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/de143497
Branch: refs/heads/master
Commit: de143497816dbeec6fd48fc0ba3e6933160fa928
Parents: 1e65c7f
Author: Alexander Rukletsov <ru...@gmail.com>
Authored: Sun Feb 21 13:49:45 2016 +0100
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Sun Feb 21 13:49:45 2016 +0100
----------------------------------------------------------------------
src/exec/exec.cpp | 15 ++++++++++-----
1 file changed, 10 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/de143497/src/exec/exec.cpp
----------------------------------------------------------------------
diff --git a/src/exec/exec.cpp b/src/exec/exec.cpp
index 83dbee9..ffa9396 100644
--- a/src/exec/exec.cpp
+++ b/src/exec/exec.cpp
@@ -646,7 +646,8 @@ Status MesosExecutorDriver::start()
// Get slave PID from environment.
value = os::getenv("MESOS_SLAVE_PID");
if (value.isNone()) {
- EXIT(1) << "Expecting 'MESOS_SLAVE_PID' to be set in the environment";
+ EXIT(EXIT_FAILURE)
+ << "Expecting 'MESOS_SLAVE_PID' to be set in the environment";
}
slave = UPID(value.get());
@@ -655,28 +656,32 @@ Status MesosExecutorDriver::start()
// Get slave ID from environment.
value = os::getenv("MESOS_SLAVE_ID");
if (value.isNone()) {
- EXIT(1) << "Expecting 'MESOS_SLAVE_ID' to be set in the environment";
+ EXIT(EXIT_FAILURE)
+ << "Expecting 'MESOS_SLAVE_ID' to be set in the environment";
}
slaveId.set_value(value.get());
// Get framework ID from environment.
value = os::getenv("MESOS_FRAMEWORK_ID");
if (value.isNone()) {
- EXIT(1) << "Expecting 'MESOS_FRAMEWORK_ID' to be set in the environment";
+ EXIT(EXIT_FAILURE)
+ << "Expecting 'MESOS_FRAMEWORK_ID' to be set in the environment";
}
frameworkId.set_value(value.get());
// Get executor ID from environment.
value = os::getenv("MESOS_EXECUTOR_ID");
if (value.isNone()) {
- EXIT(1) << "Expecting 'MESOS_EXECUTOR_ID' to be set in the environment";
+ EXIT(EXIT_FAILURE)
+ << "Expecting 'MESOS_EXECUTOR_ID' to be set in the environment";
}
executorId.set_value(value.get());
// Get working directory from environment.
value = os::getenv("MESOS_DIRECTORY");
if (value.isNone()) {
- EXIT(1) << "Expecting 'MESOS_DIRECTORY' to be set in the environment";
+ EXIT(EXIT_FAILURE)
+ << "Expecting 'MESOS_DIRECTORY' to be set in the environment";
}
workDirectory = value.get();