Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/#review148878 --- Closing this review as Benjamin has followed up with separate revi

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 26, 2016, 3:01 p.m.) Review request for mesos, Jie Yu and Kevin Kl

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 25, 2016, 6:38 p.m.) Review request for mesos, Jie Yu and Kevin Kl

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/#review133860 --- src/linux/capabilities.hpp (line 37)

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 17, 2016, 2:59 p.m.) Review request for mesos, Jie Yu and Kevin Kl

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 13, 2016, 12:45 a.m.) Review request for mesos, Jie Yu and Kevin K

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 11, 2016, 4:34 a.m.) Review request for mesos, Jie Yu and Kevin Kl

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

> On May 7, 2016, 7:14 p.m., Jie Yu wrote: > > src/linux/capabilities.hpp, line 194 > > > > > > When I review the code, I found very confusing. We have > > `CapabilityInfo, CapabilityInfo::Capability, Capability, Ca

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/#review132094 --- See my detailed comments. I think we're introducing too many class

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 6, 2016, 5:03 p.m.) Review request for mesos, Jie Yu and Kevin Klu

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated May 5, 2016, 7:42 p.m.) Review request for mesos, Jie Yu and Kevin Klu

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated April 28, 2016, 9:44 p.m.) Review request for mesos, Jie Yu and Kevin

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated April 26, 2016, 1:23 a.m.) Review request for mesos, Jie Yu and Kevin

Re: Review Request 46370: Introduced linux capabilities API.

> On April 20, 2016, 3:15 a.m., Kevin Klues wrote: > > I added a few comments below, but in general, I feel like there are places > > this code could be greatly simplified. Specifically, it's not obvious to > > me why we need all of the different classes you define (or maybe more about > > wh

Re: Review Request 46370: Introduced linux capabilities API.

> On April 20, 2016, 3:15 a.m., Kevin Klues wrote: > > src/linux/capabilities.cpp, lines 124-125 > > > > > > This should be unnecessary. See: > > https://github.com/klueska-mesosphere/mesos/blob/master/src/linux/cgr

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated April 20, 2016, 8:19 p.m.) Review request for mesos, Jie Yu and Kevin

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/ --- (Updated April 20, 2016, 7:05 p.m.) Review request for mesos, Jie Yu and Kevin

Re: Review Request 46370: Introduced linux capabilities API.

> On April 20, 2016, 3:15 a.m., Kevin Klues wrote: > > src/linux/capabilities.hpp, lines 94-99 > > > > > > From my reading of: > > http://man7.org/linux/man-pages/man7/capabilities.7.html > > > > this enum s

Re: Review Request 46370: Introduced linux capabilities API.

> On April 20, 2016, 3:15 a.m., Kevin Klues wrote: > > src/linux/capabilities.hpp, lines 32-33 > > > > > > This should all probably live in the mesos::internal::capabilities > > namespace. The reasons why I chose to

Re: Review Request 46370: Introduced linux capabilities API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46370/#review129674 --- I added a few comments below, but in general, I feel like there ar