----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47373/#review133271 -----------------------------------------------------------
Ship it! Looks good to me! src/main/java/org/apache/aurora/scheduler/reconciliation/TaskReconciler.java (line 138) <https://reviews.apache.org/r/47373/#comment197561> Is that really copy necessary? - Stephan Erb On Mai 14, 2016, 1:23 vorm., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47373/ > ----------------------------------------------------------- > > (Updated Mai 14, 2016, 1:23 vorm.) > > > Review request for Aurora, Joshua Cohen and Stephan Erb. > > > Repository: aurora > > > Description > ------- > > This helps alleviate master task reconciliation pressure in large clusters > that _may_ lead to unresponsiveness and subsequent loss of slaves given a > combination of other load factors. > > Default values are chosen somewhat randomly but have been battle tested. > Happy to consider other suggestions. > > > Diffs > ----- > > > src/main/java/org/apache/aurora/scheduler/reconciliation/ReconciliationModule.java > cccee083fc2e5f204c91a9d397beb451d4d9df40 > > src/main/java/org/apache/aurora/scheduler/reconciliation/TaskReconciler.java > 57d20618fe9eda076936d159172d5e133ec6597b > > src/test/java/org/apache/aurora/scheduler/reconciliation/TaskReconcilerTest.java > 5b4b3ac9781c9b572bbc9507c238dfa3b0177b45 > > src/test/java/org/apache/aurora/scheduler/testing/FakeScheduledExecutor.java > 9082a31d4d2ff454775f5160a474e31261628929 > > Diff: https://reviews.apache.org/r/47373/diff/ > > > Testing > ------- > > ./gradlew -Pq build > > > Thanks, > > Maxim Khutornenko > >