----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70570/#review215180 -----------------------------------------------------------
Bad review! Reviews applied: [70570] Error: 2019-05-10 09:12:39 URL:https://reviews.apache.org/r/70570/diff/raw/ [2537/2537] -> "70570.patch" [1] error: patch failed: src/master/allocator/mesos/hierarchical.cpp:1795 error: src/master/allocator/mesos/hierarchical.cpp: patch does not apply - Mesos Reviewbot On May 9, 2019, 12:22 a.m., Meng Zhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70570/ > ----------------------------------------------------------- > > (Updated May 9, 2019, 12:22 a.m.) > > > Review request for mesos and Benjamin Mahler. > > > Bugs: MESOS-9759 > https://issues.apache.org/jira/browse/MESOS-9759 > > > Repository: mesos > > > Description > ------- > > This patch logs `requiredHeadroom` and `availableHeadroom` > before each allocation cycle and logs `requiredHeadroom`, > resources and number of agents held back for quota `headroom` > as well as resources filtered at the end of each allocation > cycle. > > While we also held resources back for quota headroom in the > first stage, we do not log it there. This is because in the > second stage, we try to allocate all resources (including the > ones held back in the first stage). Thus only resources held > back in the second stage are truly held back for the whole > allocation cycle. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > 708c444ea93a63d566a81b31c4c79ed65949e1a4 > > > Diff: https://reviews.apache.org/r/70570/diff/6/ > > > Testing > ------- > > N/A > > > Thanks, > > Meng Zhu > >