You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Joseph Wu <jo...@mesosphere.io> on 2017/09/07 00:40:49 UTC

Review Request 62144: Added extra constructors for ObjectApprover::Object.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62144/
-----------------------------------------------------------

Review request for mesos, Gilbert Song and Jie Yu.


Bugs: MESOS-7492
    https://issues.apache.org/jira/browse/MESOS-7492


Repository: mesos


Description
-------

These two variants are used by the Standalone Container API,
which has a ContainerID and CommandInfo at launch, and only a
ContainerID for other APIs.


Diffs
-----

  include/mesos/authorizer/authorizer.hpp 40790f5801bdde8df0e2823fc8949d00b1eaa2fb 


Diff: https://reviews.apache.org/r/62144/diff/1/


Testing
-------

See later in chain.


Thanks,

Joseph Wu


Re: Review Request 62144: Added extra constructors for ObjectApprover::Object.

Posted by Joseph Wu <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62144/
-----------------------------------------------------------

(Updated Oct. 16, 2017, 4:46 p.m.)


Review request for mesos, Gilbert Song and Jie Yu.


Bugs: MESOS-7492
    https://issues.apache.org/jira/browse/MESOS-7492


Repository: mesos


Description
-------

These two variants are used by the Standalone Container API,
which has a ContainerID and CommandInfo at launch, and only a
ContainerID for other APIs.


Diffs (updated)
-----

  include/mesos/authorizer/authorizer.hpp 40790f5801bdde8df0e2823fc8949d00b1eaa2fb 


Diff: https://reviews.apache.org/r/62144/diff/2/

Changes: https://reviews.apache.org/r/62144/diff/1-2/


Testing
-------

See later in chain.


Thanks,

Joseph Wu