Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-10 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/ --- (Updated June 10, 2016, 9:22 p.m.) Review request for mesos, Adam B and Alexand

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-10 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/ --- (Updated June 10, 2016, 9:25 p.m.) Review request for mesos, Adam B and Alexand

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-10 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/#review137075 --- Fix it, then Ship it! LGTM! src/files/files.cpp (lines 723 -

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-10 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/#review137078 --- Patch looks great! Reviews applied: [48563, 48566] Passed comman

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-10 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/ --- (Updated June 11, 2016, 4:19 a.m.) Review request for mesos, Adam B and Alexand

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/#review137112 --- Fix it, then Ship it! LGTM, barring my nits about lambda indent

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-11 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/ --- (Updated June 11, 2016, 3:01 p.m.) Review request for mesos, Adam B and Alexand

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-11 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/ --- (Updated June 11, 2016, 3:01 p.m.) Review request for mesos, Adam B and Alexand

Re: Review Request 48566: Fixed continuation logic for sandbox authorization.

2016-06-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48566/#review137126 --- Patch looks great! Reviews applied: [48563, 48566] Passed comman