Re: Review Request 70686: Added a unit test for master operation authorization.

2019-05-21 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70686/ --- (Updated May 22, 2019, 4:09 a.m.) Review request for mesos, Benjamin Bannier, G

Re: Review Request 70686: Added a unit test for master operation authorization.

2019-05-21 Thread Chun-Hung Hsiao
> On May 21, 2019, 12:07 p.m., Benjamin Bannier wrote: > > src/tests/master_authorization_tests.cpp > > Lines 3317-3320 (patched) > > > > > > If you get rid of the `vector` `evolve` function in r/70683/ we'd > > cr

Re: Review Request 70686: Added a unit test for master operation authorization.

2019-05-21 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70686/#review215405 --- Fix it, then Ship it! This is a very exhaustive test, but sligh

Review Request 70686: Added a unit test for master operation authorization.

2019-05-20 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70686/ --- Review request for mesos, Benjamin Bannier and Greg Mann. Bugs: MESOS-9485