You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gr...@apache.org on 2018/02/01 06:20:42 UTC
[06/12] mesos git commit: Made `master::receive()` drop
`ACKNOWLEDGE_OPERATION_STATUS` calls.
Made `master::receive()` drop `ACKNOWLEDGE_OPERATION_STATUS` calls.
`Master::receive()` handles scheduler API calls sent by v0 API clients.
`ACKNOWLEDGE_OPERATION_STATUS` is a v1-only API call, so
`Master::receive()` should drop it.
Review: https://reviews.apache.org/r/65360/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/84ce9fda
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/84ce9fda
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/84ce9fda
Branch: refs/heads/master
Commit: 84ce9fda3c2d5885f2b4b99e62e0a32e851cf4b8
Parents: fccdba9
Author: Gaston Kleiman <ga...@mesosphere.io>
Authored: Wed Jan 31 12:59:38 2018 -0800
Committer: Greg Mann <gr...@gmail.com>
Committed: Wed Jan 31 18:14:23 2018 -0800
----------------------------------------------------------------------
src/master/master.cpp | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/84ce9fda/src/master/master.cpp
----------------------------------------------------------------------
diff --git a/src/master/master.cpp b/src/master/master.cpp
index 35f4d6f..fdbbb16 100644
--- a/src/master/master.cpp
+++ b/src/master/master.cpp
@@ -2419,9 +2419,10 @@ void Master::receive(
}
case scheduler::Call::ACKNOWLEDGE_OPERATION_STATUS: {
- acknowledgeOperationStatus(
- framework,
- call.acknowledge_operation_status());
+ drop(
+ from,
+ call,
+ "'ACKNOWLEDGE_OPERATION_STATUS' is not supported by the v0 API");
break;
}