Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-12-01 Thread Jiang Yan Xu
> On Nov. 21, 2017, 10:27 a.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.cpp > > Lines 430-437 (original), 431-438 (patched) > > > > > > Do we need a lambda here? > > Jiang Yan Xu

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-12-01 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63831/#review192506 --- Ship it! Ship It! - Alexander Rukletsov On Nov. 28, 2017,

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-12-01 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63831/#review192498 --- FAIL: Mesos libprocess-tests failed to build Reviews applied:

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-12-01 Thread Alexander Rukletsov
> On Nov. 21, 2017, 6:27 p.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.cpp > > Lines 430-437 (original), 431-438 (patched) > > > > > > Do we need a lambda here? > > Jiang Yan Xu

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-11-27 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63831/ --- (Updated Nov. 27, 2017, 4:45 p.m.) Review request for mesos and Alexander

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-11-27 Thread Jiang Yan Xu
> On Nov. 21, 2017, 10:27 a.m., Alexander Rukletsov wrote: > > src/master/allocator/mesos/hierarchical.cpp > > Lines 430-437 (original), 431-438 (patched) > > > > > > Do we need a lambda here? I kept the original

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-11-21 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63831/#review191613 --- src/master/allocator/mesos/hierarchical.cpp Lines 430-437

Re: Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-11-16 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63831/#review191165 --- PASS: Mesos patch 63831 was successfully built and tested.

Review Request 63831: Fixed a bug that removed the suppressed framework from sorter.

2017-11-15 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63831/ --- Review request for mesos and Alexander Rukletsov. Bugs: MESOS-8223