Re: Review Request 64741: Eliminated some unnecessary copying in the HTTP operator API.

2017-12-20 Thread Benjamin Mahler
> On Dec. 20, 2017, 7:52 p.m., Meng Zhu wrote: > > src/master/http.cpp > > Lines 4232-4235 (original), 4232-4234 (patched) > > > > > > how about: > > > > `*getTasks.add_pending_tasks() =

Re: Review Request 64741: Eliminated some unnecessary copying in the HTTP operator API.

2017-12-20 Thread Meng Zhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64741/#review194280 --- Ship it! Ship It! - Meng Zhu On Dec. 19, 2017, 5:47 p.m.,

Re: Review Request 64741: Eliminated some unnecessary copying in the HTTP operator API.

2017-12-20 Thread Meng Zhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64741/#review194269 --- src/master/http.cpp Lines 4232-4235 (original), 4232-4234

Re: Review Request 64741: Eliminated some unnecessary copying in the HTTP operator API.

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

Review Request 64741: Eliminated some unnecessary copying in the HTTP operator API.

2017-12-19 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64741/ --- Review request for mesos, Michael Park and Meng Zhu. Bugs: MESOS-8344