----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54337/#review157910 -----------------------------------------------------------
Ship it! Ship It! - Vinod Kone On Dec. 3, 2016, 7:19 p.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54337/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2016, 7:19 p.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 > >
