Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-15 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/#review129229 --- Ship it! Ship It! - Jie Yu On April 16, 2016, 12:10 a.m.,

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-15 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/ --- (Updated April 15, 2016, 8:10 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-15 Thread Kapil Arya
> On April 14, 2016, 1:58 p.m., Jie Yu wrote: > > include/mesos/state/state.hpp, line 83 > > > > > > Let's void mesos:: if possible. Ditto for others. Some of the `mesos::`s are required due to conflicts with

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-15 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/ --- (Updated April 15, 2016, 7 p.m.) Review request for mesos and Jie Yu. Bugs:

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-15 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/ --- (Updated April 15, 2016, 6:17 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-15 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/ --- (Updated April 15, 2016, 6:12 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-14 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/#review128935 --- include/mesos/state/state.hpp (line 82)

Re: Review Request 46164: Moved mesos::internal::state to mesos::state namespace.

2016-04-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46164/#review128797 --- Bad review! Reviews applied: [] Error: No reviewers specified.