Re: Review Request 61538: Used common port range interval code in the port_mapping isolator.

2017-08-22 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61538/#review183511 --- Fix it, then Ship it!

Re: Review Request 61538: Used common port range interval code in the port_mapping isolator.

2017-08-22 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61538/ --- (Updated Aug. 22, 2017, 6:56 p.m.) Review request for mesos, Qian Zhang and

Re: Review Request 61538: Used common port range interval code in the port_mapping isolator.

2017-08-22 Thread James Peach
> On Aug. 22, 2017, 3:45 a.m., Jie Yu wrote: > > Can you explain a bit more why this change? This was [requested](https://reviews.apache.org/r/60492/#comment257154) in a subsequent review. As per our slack conversation, I'll look into making this work for `IntervalSet`. - James

Re: Review Request 61538: Used common port range interval code in the port_mapping isolator.

2017-08-21 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61538/#review183439 --- Can you explain a bit more why this change? - Jie Yu On Aug.

Review Request 61538: Used common port range interval code in the port_mapping isolator.

2017-08-09 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61538/ --- Review request for mesos, Qian Zhang and Jiang Yan Xu. Bugs: MESOS-7675