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

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. - M

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 Toenshoff.

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

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

2015-05-20 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/#review84497 --- Ship it! I admit I don't like namespace aliases which have the scop

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 consid

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. - M

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.,

Review Request 34375: Removed use of namespace aliases.

2015-05-18 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/ --- Review request for mesos. Repository: mesos Description --- See summary.