Re: Review Request 34375: Removed use of namespace aliases.

2015-06-15 Thread Ben Mahler
On May 19, 2015, 2:02 a.m., Ben Mahler wrote: This might be a valid namespace alias use case that we hadn't considered, because there is no way to be able to write just `http::Response` otherwise, is there? Seems quite verbose to write process::http everywhere, and on the otherhand

Re: Review Request 34375: Removed use of namespace aliases.

2015-06-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/#review87851 --- Patch looks great! Reviews applied: [34375] All tests passed. -

Re: Review Request 34375: Removed use of namespace aliases.

2015-06-13 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/ --- (Updated June 13, 2015, 9:51 p.m.) Review request for mesos and Till

Re: Review Request 34375: Removed use of namespace aliases.

2015-05-23 Thread Michael Park
On May 19, 2015, 2:02 a.m., Ben Mahler wrote: This might be a valid namespace alias use case that we hadn't considered, because there is no way to be able to write just `http::Response` otherwise, is there? Seems quite verbose to write process::http everywhere, and on the otherhand

Re: Review Request 34375: Removed use of namespace aliases.

2015-05-18 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/#review84238 --- Ship it! Ship It! - Till Toenshoff On May 18, 2015, 11:13 p.m.,

Re: Review Request 34375: Removed use of namespace aliases.

2015-05-18 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/#review84247 --- Patch looks great! Reviews applied: [34375] All tests passed. -

Re: Review Request 34375: Removed use of namespace aliases.

2015-05-18 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/#review84251 --- This might be a valid namespace alias use case that we hadn't