Re: Review Request 63271: Windows: Added `os::set_job_memory_limit` to stout.

2017-11-10 Thread Akash Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63271/#review190771 --- Ship it! Ship It! - Akash Gupta On Nov. 2, 2017, 8:39 p.m.,

Re: Review Request 63271: Windows: Added `os::set_job_memory_limit` to stout.

2017-10-26 Thread Jeff Coffler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63271/#review189347 --- Ship it! Ship It! - Jeff Coffler On Oct. 26, 2017, 4:29

Re: Review Request 63271: Windows: Added `os::set_job_memory_limit` to stout.

2017-10-26 Thread Andrew Schwartzmeyer
> On Oct. 26, 2017, 1:55 p.m., Jeff Coffler wrote: > > 3rdparty/stout/include/stout/windows/os.hpp > > Lines 784 (patched) > > > > > > I usually see NOLINT on a line of it's own. Is that a mistake, making > > the

Re: Review Request 63271: Windows: Added `os::set_job_memory_limit` to stout.

2017-10-26 Thread Jeff Coffler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63271/#review189342 --- 3rdparty/stout/include/stout/windows/os.hpp Lines 784 (patched)

Review Request 63271: Windows: Added `os::set_job_memory_limit` to stout.

2017-10-26 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63271/ --- Review request for mesos, Akash Gupta, Jeff Coffler, Jie Yu, John Kordich, and