----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61918/#review184005 -----------------------------------------------------------
src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java Line 357 (original), 360 (patched) <https://reviews.apache.org/r/61918/#comment260027> Drop the `FluentIterable.from(` since it is againg getting wrapped into the same below. src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java Line 361 (original), 367 (patched) <https://reviews.apache.org/r/61918/#comment260028> How about this method? - Santhosh Kumar Shanmugham On Aug. 25, 2017, 1:31 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61918/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2017, 1:31 p.m.) > > > Review request for Aurora, David McLaughlin, Dmitry Zhuk, Santhosh Kumar > Shanmugham, and Stephan Erb. > > > Repository: aurora > > > Description > ------- > > Currently, `getOffers` and `getWeaklyConsistentOffers` provide iterables that > use lambdas (in filter) not backed by concurrent data structures > (`globallyBannedOffers` and `staticallyBannedOffers`). This can lead to a > race condition where we have unknown behavior if reading and modification > happens at the same time. > > To fix this, in `getOffers` we will revert to the previous behavior (before > patch https://reviews.apache.org/r/61804/) where we create a copy of the > current offers as well as a copy of the banned offers for consistency. > > For `getWeaklyConsistentOffers`, we will use concurrent data structures for > `globallyBannedOffers` and `staticallyBannedOffers` so we don't have a race > that may produce unknown side effects. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/offers/OfferManager.java > 5697d2e2eb001042511924442ff2dbe358451642 > > > Diff: https://reviews.apache.org/r/61918/diff/1/ > > > Testing > ------- > > Unit tests pass. > > Currently running end to end test and testing on a live cluster. > > > Thanks, > > Jordan Ly > >
