> On Dec. 14, 2015, 2:14 a.m., Klaus Ma wrote: > > src/master/allocator/mesos/hierarchical.cpp, line 431 > > <https://reviews.apache.org/r/40632/diff/6/?file=1161883#file1161883line431> > > > > Honestly, I'd like to separate optimistic resources into other counter > > in allocator; it's a litte different with oversubscription: here, one > > reserved resources are counting twice in total. > > Guangya Liu wrote: > The allocation slack was decreased in 1304-1329
Just wondering the reason to set resources in allocaction slack when add/update slave. It seems to me that you just need to calculate available resources in allocate() and add allocation slack to the available there? - Jian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40632/#review110154 ----------------------------------------------------------- On Dec. 13, 2015, 11:35 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40632/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2015, 11:35 a.m.) > > > Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, Joseph Wu, > and Klaus Ma. > > > Repository: mesos > > > Description > ------- > > WIP: Enabled oversubscribed resources for reservations in allocator. > > > Diffs > ----- > > src/master/allocator/mesos/hierarchical.cpp > 5da825a1d578a9ee40b4985378fddb3c5fb3b416 > > Diff: https://reviews.apache.org/r/40632/diff/ > > > Testing > ------- > > > Thanks, > > Guangya Liu > >
