Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers

2017-08-28 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61918/#review184012 --- Ship it! Ship It! - Santhosh Kumar Shanmugham On Aug. 25,

Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers

2017-08-28 Thread Santhosh Kumar Shanmugham
> On Aug. 28, 2017, 4:08 p.m., Santhosh Kumar Shanmugham wrote: > > src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java > > Line 361 (original), 367 (patched) > > > > > > How about this method? > >

Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers

2017-08-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61918/#review184010 --- Master (364eac5) is green with this patch.

Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers

2017-08-28 Thread Jordan Ly
> On Aug. 28, 2017, 11:08 p.m., Santhosh Kumar Shanmugham wrote: > > src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java > > Line 361 (original), 367 (patched) > > > > > > How about this method? I

Re: Review Request 61918: Fix concurrency issues around banned offers in HostOffers

2017-08-28 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61918/#review184005 ---