Re: Review Request 40559: Added a wait() function to Subprocess.

2015-12-03 Thread Jie Yu
> On Dec. 3, 2015, 10:34 p.m., Vinod Kone wrote: > > Ship It! There's some controversy regarding this patch. I'll move this logic to hdfs.cpp as a helper. Discard this patch for now. Will send a new one shortly. - Jie --- This is an au

Re: Review Request 40559: Added a wait() function to Subprocess.

2015-12-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40559/#review108898 --- Ship it! Ship It! - Vinod Kone On Nov. 20, 2015, 9:06 p.m., Jie

Re: Review Request 40559: Added a wait() function to Subprocess.

2015-11-20 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40559/#review107475 --- Patch looks great! Reviews applied: [40250, 40251, 40252, 40274, 4

Re: Review Request 40559: Added a wait() function to Subprocess.

2015-11-20 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40559/ --- (Updated Nov. 20, 2015, 9:06 p.m.) Review request for mesos, Ben Mahler and Vin