Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-16 Thread Cong Wang
> On May 14, 2016, 2:01 a.m., Qian Zhang wrote: > > src/slave/slave.cpp, line 1020 > > > > > > If we do the link here, that means we still establish TCP connection > > with master right after a new master is

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-15 Thread Qian Zhang
> On May 14, 2016, 10:01 a.m., Qian Zhang wrote: > > src/slave/slave.cpp, line 1020 > > > > > > If we do the link here, that means we still establish TCP connection > > with master right after a new master is

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-14 Thread Cong Wang
> On May 14, 2016, 2:01 a.m., Qian Zhang wrote: > > src/slave/slave.cpp, line 1020 > > > > > > If we do the link here, that means we still establish TCP connection > > with master right after a new master is

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-13 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/#review133243 --- src/slave/slave.cpp (line 1019)

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-12 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/#review133025 --- Ship it! Ship It! - Ben Mahler On May 13, 2016, 3:37 a.m.,

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-12 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/ --- (Updated May 13, 2016, 3:37 a.m.) Review request for mesos, Ben Mahler, Ian

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-12 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/#review133016 --- src/slave/slave.cpp (line 996)

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-12 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/ --- (Updated May 12, 2016, 6:18 p.m.) Review request for mesos, Ben Mahler, Ian

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-12 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/#review132956 --- src/slave/slave.cpp (line 966)

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/#review132638 --- Patch looks great! Reviews applied: [47209] Passed command:

Re: Review Request 47209: Establish TCP connection after backing off.

2016-05-10 Thread Cong Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47209/#review132597 --- Ship it! Ship It! - Cong Wang On May 11, 2016, 12:58 a.m.,