----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37969/ -----------------------------------------------------------
Review request for mesos, Benjamin Hindman, Artem Harutyunyan, and Joris Van Remoortere. Repository: mesos Description ------- Replaces `__.DebugString()` with `stringify(JSON::Protobuf(__))`, which looks nicer and matches the JSON expected by the HTTP endpoints. Addresses a comment found here: https://reviews.apache.org/r/37358/#comment152513 Diffs ----- src/master/http.cpp 94e97a2898106579434e8cdec04b7b0e130a810e src/master/maintenance.cpp 277dd8270fbd497616e5b24db4ea7dd615e170e3 Diff: https://reviews.apache.org/r/37969/diff/ Testing ------- `make check` This change will tweak the output from: ``` Machine 'hostname: "myComputer" ' is not valid because ... ``` (`DebugString` ^ puts a newline there.) To: ``` Machine '{"hostname":"myComputer"}' is not valid because ... ``` Thanks, Joseph Wu
