Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-06 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/ --- (Updated May 6, 2016, 9:20 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-06 Thread Greg Mann
> On May 6, 2016, 6:01 p.m., Kapil Arya wrote: > > 3rdparty/libprocess/src/process.cpp, lines 3346-3355 > > > > > > Can we simplify the logic a bit here and add some explanation? This review was merged into https:/

Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-06 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/#review132085 --- 3rdparty/libprocess/src/process.cpp (lines 3299 - 3308)

Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-05 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/ --- (Updated May 5, 2016, 8:15 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-05 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/ --- (Updated May 5, 2016, 6:58 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-05 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/ --- (Updated May 5, 2016, 1:46 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-04 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/ --- (Updated May 4, 2016, 9:21 p.m.) Review request for mesos, Alexander Rojas and

Review Request 46989: Added authorization handling to 'ProcessBase::visit'.

2016-05-04 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46989/ --- Review request for mesos, Alexander Rojas and Kapil Arya. Bugs: MESOS-5286