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

(Updated Jan. 15, 2014, 1:42 a.m.)


Review request for mesos, Benjamin Hindman, Ben Mahler, Jeff Currier, and Jie 
Yu.


Changes
-------

  Incorporate CR feedback.

    Renamed internal::httpRequest to just be internal::request.

    Changed internal::request to take Option<string> so that None
    indicates no body / query instead of empty string.


Bugs: https://issues.apache.org/jira/browse/MESOS-902
    
https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/MESOS-902


Repository: mesos-git


Description
-------

   Add post function to http.hpp of libprocess.

    This adds the post equivalent of the get function to http.hpp.

    The existing get method is refactored into internal::httpRequest
    with arguments for method, query, and body.

    The get and post functions can then be implemented as simple
    wrapper of internal::httpRequest.

    There are also new unit tests to verify the existing behavior of
    get and the new post behavior.

    See:  https://issues.apache.org/jira/browse/MESOS-902

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


Diffs (updated)
-----

  3rdparty/libprocess/src/process.cpp 67f7f9b 

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


Testing
-------

added new unit tests
make check


Thanks,

Charlie Carson

Reply via email to