You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Niklas Nielsen <ni...@qni.dk> on 2014/11/20 20:31:47 UTC

Re: Review Request 28287: Simplify JSON modeling with JSON::Protobuf

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28287/
-----------------------------------------------------------

(Updated Nov. 20, 2014, 11:31 a.m.)


Review request for mesos, Ben Mahler and Timothy Chen.


Summary (updated)
-----------------

Simplify JSON modeling with JSON::Protobuf


Repository: mesos-git


Description
-------

JSON fields doesn't need to be set explicitly, so label modeling can be simplified with JSON::Protobuf.


Diffs (updated)
-----

  src/common/http.cpp b21e18d547d716e540dc2ce47b2cb1ab898bce94 

Diff: https://reviews.apache.org/r/28287/diff/


Testing
-------

make check


Thanks,

Niklas Nielsen


Re: Review Request 28287: Simplify Label JSON modeling with JSON::Protobuf

Posted by Niklas Nielsen <ni...@qni.dk>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28287/
-----------------------------------------------------------

(Updated Nov. 20, 2014, 11:32 a.m.)


Review request for mesos, Ben Mahler and Timothy Chen.


Changes
-------

Sorry for the noise - changing to original title.


Summary (updated)
-----------------

Simplify Label JSON modeling with JSON::Protobuf


Repository: mesos-git


Description
-------

JSON fields doesn't need to be set explicitly, so label modeling can be simplified with JSON::Protobuf.


Diffs
-----

  src/common/http.cpp b21e18d547d716e540dc2ce47b2cb1ab898bce94 

Diff: https://reviews.apache.org/r/28287/diff/


Testing
-------

make check


Thanks,

Niklas Nielsen