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

Advertising

(Updated Aug. 5, 2015, 6:05 a.m.)
Review request for mesos, Ben Mahler and Jie Yu.
Changes
-------
* Added a comment
* Removed the implicit empty string check
Summary (updated)
-----------------
Extended 'getFormValue' to 'getFormValues' in order to parse input uniformly.
Repository: mesos
Description (updated)
-------
`getFormValues` takes a `Request` object and a list of required keys, and tries
to return a map of the required keys to their corresponding values.
This function helps to reduce the amount of code required to check the each
required keys manually as well as to keep the missing parameter error message
uniform across multiple endpoints.
This is a short-term solution which improves upon `getFormValue`, but in the
future we should consider adding a libprocess level support for processing the
required and optional keys from a `Request`.
Diffs (updated)
-----
src/master/http.cpp 76e70801925041f08bc94f0ca18c86f1a573b2b3
src/tests/repair_tests.cpp 4d513ed9e18ebf04a5e5aadb022b064479739e66
Diff: https://reviews.apache.org/r/36987/diff/
Testing
-------
`make check`
Thanks,
Michael Park