> On April 7, 2016, 11:07 a.m., Joerg Schad wrote: > > src/master/weights_handler.cpp, line 90 > > <https://reviews.apache.org/r/45863/diff/1/?file=1329646#file1329646line90> > > > > Not yours: Above the request.body is included in the output. Feels > > inconsistent here... > > Alexander Rukletsov wrote: > Above are errors for the whole request, here they are per role basis.
Ok, (unrelated to this) would you then say we should to remove in for quota_handler as well? ```return BadRequest( "Failed to validate set quota request JSON '" + request.body + "': Unknown role '" + quotaInfo.role() + "'"); }``` - Joerg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45863/#review127580 ----------------------------------------------------------- On April 7, 2016, 10:50 a.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45863/ > ----------------------------------------------------------- > > (Updated April 7, 2016, 10:50 a.m.) > > > Review request for mesos, Yongqiao Wang and Joerg Schad. > > > Repository: mesos > > > Description > ------- > > While writing log or error messages we adhere the following style: > * No period at the end (there are exceptions though, e.g., one is > when constructing request responses. > * When splitting a string over multiple lines, put a space at the > beginning of the following line in contrast to the end of the > previous line. > > > Diffs > ----- > > src/master/weights_handler.cpp e88bf2ab67ccadf35879b92f3280298a43d7cd0e > > Diff: https://reviews.apache.org/r/45863/diff/ > > > Testing > ------- > > On Mac OS 10.10.4: > `make check` > > > Thanks, > > Alexander Rukletsov > >