----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54337/#review157897 -----------------------------------------------------------
src/tests/api_tests.cpp (line 3828) <https://reviews.apache.org/r/54337/#comment228520> hmm. InternalServerError instead of a BadRequest sounds wrong. for non-streaming requests, looks like validation error results in BadRequest(); but not so for streaming requests? - Vinod Kone On Dec. 3, 2016, 6:01 a.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54337/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2016, 6:01 a.m.) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Previously, we were validating that all records on the stream > should have 'attach_container_input.container_id' set. However, > only the first record should have it set and be of type > 'AttachContainerInput::CONTAINER_ID'. > > This also raises the question if we should perform this > validation in the handler itself for streaming requests. > > > Diffs > ----- > > src/slave/http.cpp 8b104ce9ca6e72a1e76943f0d68a0541a45e39f6 > src/slave/validation.cpp 15330ad6e63cf6adbdf36a3c1d1a60d2315ae511 > src/tests/api_tests.cpp afae6a7e0809174f48f280f170fad9315e80a906 > > Diff: https://reviews.apache.org/r/54337/diff/ > > > Testing > ------- > > make check + added test > > > Thanks, > > Anand Mazumdar > >
