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/01/17 01:08:52 UTC

mesos git commit: Used jsonify in http serialization for better performance.

Repository: mesos
Updated Branches:
  refs/heads/master 9b1fda56a -> aded41aec


Used jsonify in http serialization for better performance.

This change avoids building an in-memory JSON::Object in favor
of using jsonify to directly output the serialized JSON from
the protobuf.

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


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

Branch: refs/heads/master
Commit: aded41aecb5aab988313689c2b0664b86d45eb1f
Parents: 9b1fda5
Author: Meng Zhu <mz...@mesosphere.io>
Authored: Tue Jan 16 17:08:05 2018 -0800
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Tue Jan 16 17:08:05 2018 -0800

----------------------------------------------------------------------
 src/common/http.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/aded41ae/src/common/http.cpp
----------------------------------------------------------------------
diff --git a/src/common/http.cpp b/src/common/http.cpp
index 3ada1f0..728fc55 100644
--- a/src/common/http.cpp
+++ b/src/common/http.cpp
@@ -114,7 +114,7 @@ string serialize(
       return message.SerializeAsString();
     }
     case ContentType::JSON: {
-      return stringify(JSON::protobuf(message));
+      return jsonify(JSON::Protobuf(message));
     }
     case ContentType::RECORDIO: {
       LOG(FATAL) << "Serializing a RecordIO stream is not supported";