----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36847/#review93880 -----------------------------------------------------------
3rdparty/libprocess/include/process/http.hpp (line 736) <https://reviews.apache.org/r/36847/#comment148297> (We have discussed this naming choice at some length offline. Here is how I see it.) If I find a call site somewhere that reads "http:del(...)" I will probably wonder what this is and will look it up. Not good enough IMHO. Cryptic. If instead I read "http::sendDeleteRequest()", it is immediately clear to me what this call does. True, this makes the API unbalanced, since the other calls are simply called "get" and "put". Well, they never should have! Even if we never rename the other calls, I favor readability at call sites over superficially consistent naming across the API. 2c - Bernd Mathiske On July 29, 2015, 6:39 a.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36847/ > ----------------------------------------------------------- > > (Updated July 29, 2015, 6:39 a.m.) > > > Review request for mesos, Alexander Rukletsov and Bernd Mathiske. > > > Bugs: MESOS-3152 > https://issues.apache.org/jira/browse/MESOS-3152 > > > Repository: mesos > > > Description > ------- > > see summary. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/http.hpp > 9faed55247a3ccd629db7b85dbf31d3117e120e9 > 3rdparty/libprocess/src/http.cpp d1685799f4c53e067d0812e037e171324ee7195f > 3rdparty/libprocess/src/tests/http_tests.cpp > 01f243cd9c46e162c16e9bb452a846faf31d1445 > > Diff: https://reviews.apache.org/r/36847/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Joerg Schad > >