> On Dec. 8, 2015, 7:08 a.m., Timothy Chen wrote: > > Can you elaborate more without this fix what happened and why?
This fix moves "Accept" call before sending a response back to the client so that when the client sends the next request, the server is ready to accept it. Before this fix, there was a race condition between server sending the response and client sending the next request and server missing that request because the "accept" was not called yet. - Jojy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41078/#review109288 ----------------------------------------------------------- On Dec. 8, 2015, 7:03 a.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41078/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2015, 7:03 a.m.) > > > Review request for mesos and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > By calling accept before sending a response, the server would be ready for > client's next request. > > > Diffs > ----- > > src/tests/containerizer/provisioner_docker_tests.cpp > c63bf53fee40ef12536a16e11f4d5224c4e4278e > > Diff: https://reviews.apache.org/r/41078/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Jojy Varghese > >
