Repository: mesos
Updated Branches:
  refs/heads/master 5e4c596ac -> b10c9590a


Removed binary 'data' fields from state endpoints for memory scalability.

Review: https://reviews.apache.org/r/39611


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/b10c9590
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/b10c9590
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/b10c9590

Branch: refs/heads/master
Commit: b10c9590ab1480d78b9fef09450c993260ed7dac
Parents: 5e4c596
Author: Joseph Wu <[email protected]>
Authored: Wed Nov 4 16:38:31 2015 -0800
Committer: Benjamin Mahler <[email protected]>
Committed: Wed Nov 4 16:38:31 2015 -0800

----------------------------------------------------------------------
 docs/upgrades.md    | 5 +++++
 src/common/http.cpp | 1 -
 src/slave/http.cpp  | 1 -
 3 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/b10c9590/docs/upgrades.md
----------------------------------------------------------------------
diff --git a/docs/upgrades.md b/docs/upgrades.md
index c7bf6a2..045b018 100644
--- a/docs/upgrades.md
+++ b/docs/upgrades.md
@@ -15,6 +15,11 @@ This document serves as a guide for users who wish to 
upgrade an existing mesos
 
 **NOTE** `Credential` protobuf was changed. `Credential` field `secret` is now 
a string, it used to be bytes. This will affect framework developers and 
language bindings ought to update their generated protobuf with the new 
version. This fixes JSON based credentials file support.
 
+**NOTE** The `/state` endpoints on master and slave will no longer include 
`data` fields as part of the JSON models for `ExecutorInfo` and `TaskInfo` out 
of consideration for memory scalability (see 
[MESOS-3794](https://issues.apache.org/jira/browse/MESOS-3794) and [this email 
thread](http://www.mail-archive.com/[email protected]/msg33536.html)).
+On master, the affected `data` field was originally found via 
`frameworks[*].executors[*].data`.
+On slaves, the affected `data` field was originally found via 
`executors[*].tasks[*].data`.
+
+
 ## Upgrading from 0.24.x to 0.25.x
 
 **NOTE** The following endpoints will be deprecated in favor of new endpoints. 
Both versions will be available in 0.25 but the deprecated endpoints will be 
removed in a subsequent release.

http://git-wip-us.apache.org/repos/asf/mesos/blob/b10c9590/src/common/http.cpp
----------------------------------------------------------------------
diff --git a/src/common/http.cpp b/src/common/http.cpp
index abb4c6b..3b32b8c 100644
--- a/src/common/http.cpp
+++ b/src/common/http.cpp
@@ -323,7 +323,6 @@ JSON::Object model(const ExecutorInfo& executorInfo)
   JSON::Object object;
   object.values["executor_id"] = executorInfo.executor_id().value();
   object.values["name"] = executorInfo.name();
-  object.values["data"] = executorInfo.data();
   object.values["framework_id"] = executorInfo.framework_id().value();
   object.values["command"] = model(executorInfo.command());
   object.values["resources"] = model(executorInfo.resources());

http://git-wip-us.apache.org/repos/asf/mesos/blob/b10c9590/src/slave/http.cpp
----------------------------------------------------------------------
diff --git a/src/slave/http.cpp b/src/slave/http.cpp
index d6df97f..ce48a05 100644
--- a/src/slave/http.cpp
+++ b/src/slave/http.cpp
@@ -101,7 +101,6 @@ JSON::Object model(const TaskInfo& task)
   object.values["name"] = task.name();
   object.values["slave_id"] = task.slave_id().value();
   object.values["resources"] = model(task.resources());
-  object.values["data"] = task.data();
 
   if (task.has_command()) {
     object.values["command"] = model(task.command());

Reply via email to