Re: Review Request 43661: Added check to not Subscribe if we are already subscribed.

2016-02-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43661/#review121154 --- Ship it! Ship It! - Vinod Kone On Feb. 17, 2016, 7:22 p.m.,

Re: Review Request 43661: Added check to not Subscribe if we are already subscribed.

2016-02-23 Thread Anand Mazumdar
> On Feb. 23, 2016, 10:17 p.m., Vinod Kone wrote: > > src/scheduler/scheduler.cpp, line 216 > > > > > > Subscribe contains FrameworkInfo. So this check means that a framework > > cannot update its FrameworkInfo unle

Re: Review Request 43661: Added check to not Subscribe if we are already subscribed.

2016-02-23 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43661/#review120398 --- src/scheduler/scheduler.cpp (line 216)