Re: Review Request 65842: Changed `os::spawn` to return `Option` instead of `int`.

2018-02-28 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65842/#review198433 --- Patch looks great! Reviews applied: [65839, 65840, 65841, 65842]

Re: Review Request 65842: Changed `os::spawn` to return `Option` instead of `int`.

2018-02-28 Thread Andrew Schwartzmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65842/#review198424 --- Ditto, can you split this review per-project? I _highly_

Re: Review Request 65842: Changed `os::spawn` to return `Option` instead of `int`.

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

Review Request 65842: Changed `os::spawn` to return `Option` instead of `int`.

2018-02-28 Thread Akash Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65842/ --- Review request for mesos, Andrew Schwartzmeyer, Joseph Wu, and Michael Park.