Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-04-16 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/#review201195 --- Patch looks great! Reviews applied: [66038, 66039] Passed

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-04-14 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/#review201162 --- FAIL: Some of the unit tests failed. Please check the relevant

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-04-14 Thread fei long
> On April 4, 2018, 5:50 a.m., Gilbert Song wrote: > > src/slave/slave.cpp > > Lines 6743 (patched) > > > > > > I would leave this to @andschwa since I suspect we could have a windows > > implementation together.

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-04-14 Thread fei long
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/ --- (Updated April 14, 2018, 11:02 a.m.) Review request for mesos and Gilbert

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-04-03 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/#review200429 --- src/slave/slave.cpp Lines 6743 (patched)

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-03-13 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/#review199101 --- Patch looks great! Reviews applied: [66038, 66039] Passed

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-03-13 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/#review199093 --- PASS: Mesos patch 66039 was successfully built and tested.

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-03-13 Thread fei long
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/ --- (Updated March 13, 2018, 2:05 p.m.) Review request for mesos and Gilbert Song.

Re: Review Request 66039: Check both disk and inode usage when slaves schedule gc.

2018-03-13 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66039/#review199083 --- FAIL: Some of the unit tests failed. Please check the relevant