Re: Review Request 41078: Fixed tests to call socket accept before sending response.

2015-12-10 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41078/#review109717 --- Ship it! Ship It! - Timothy Chen On Dec. 8, 2015, 7:03 a.m.,

Re: Review Request 41078: Fixed tests to call socket accept before sending response.

2015-12-08 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41078/#review109306 --- Patch looks great! Reviews applied: [40872, 40873, 41078] Passed

Re: Review Request 41078: Fixed tests to call socket accept before sending response.

2015-12-07 Thread Jojy Varghese
> 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,

Re: Review Request 41078: Fixed tests to call socket accept before sending response.

2015-12-07 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41078/#review109288 --- Can you elaborate more without this fix what happened and why? -