Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-27 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/ --- (Updated Feb. 27, 2016, 10:30 a.m.) Review request for mesos and Vinod Kone.

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-26 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/ --- (Updated Feb. 27, 2016, 4:43 a.m.) Review request for mesos and Vinod Kone.

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-26 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/#review121046 --- Fix it, then Ship it! src/scheduler/scheduler.cpp (line 276)

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-26 Thread Anand Mazumdar
> On Feb. 26, 2016, 8:29 p.m., Vinod Kone wrote: > > src/scheduler/scheduler.cpp, line 394 > > > > > > what if we are in CONNECTING state? Good catch. I had missed that. Added a explicit check for CONNECTING too.

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-26 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/ --- (Updated Feb. 27, 2016, 1:22 a.m.) Review request for mesos and Vinod Kone.

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-26 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/#review120947 --- src/scheduler/scheduler.cpp (lines 203 - 206)

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-25 Thread Anand Mazumdar
> On Feb. 24, 2016, 3:03 a.m., Vinod Kone wrote: > > src/scheduler/scheduler.cpp, lines 239-244 > > > > > > Shouldn't we do the same for executor library? > > > > More importnatly, what if a previous

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-25 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/ --- (Updated Feb. 25, 2016, 4:27 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-23 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/#review120426 --- src/scheduler/scheduler.cpp (lines 117 - 119)

Re: Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-22 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/ --- (Updated Feb. 22, 2016, 8:19 p.m.) Review request for mesos and Vinod Kone.

Review Request 43662: Added support for pipelining calls to the scheduler library.

2016-02-17 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43662/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-3570