Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-12 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/#review118990 --- Ship it! 3rdparty/libprocess/src/tests/http_tests.cpp (lines 2

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/#review118838 --- Patch looks great! Reviews applied: [43469, 39968] Passed comman

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-10 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/#review118819 --- Ship it! Ship It! - haosdent huang On Feb. 11, 2016, 5:37 a.

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-10 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/ --- (Updated 二月 11, 2016, 5:37 a.m.) Review request for mesos and Ben Mahler. Bug

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-10 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/#review118727 --- Patch looks great! Reviews applied: [39968] Passed command: expo

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-10 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/#review118671 --- 3rdparty/libprocess/src/help.cpp (lines 170 - 187)

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-10 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39968/ --- (Updated 二月 10, 2016, 1:37 p.m.) Review request for mesos and Ben Mahler. Sum

Re: Review Request 39968: Enabled http endpoint include nested paths.

2016-02-10 Thread Guangya Liu
> On 二月 10, 2016, 6:47 a.m., haosdent huang wrote: > > 3rdparty/libprocess/src/help.cpp, line 124 > > > > > > I think could use > > ``` > > strings::split(strings::trim(request.url.path, strings::ANY, "/"), "