> On July 12, 2015, 5:16 a.m., Kapil Arya wrote:
> > It looks like it was introduced in f511395e instead (unless I am missing 
> > something here):
> > 
> > https://github.com/apache/mesos/commit/f511395e#diff-c8ca6e064a8bf7b1b3c70e6525eabeceR129

That's what I'm seeing too. Do you still want to remove this? It seems like 
valid code unless we're able to auto-reorder volume mounts?


- Timothy


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


On July 12, 2015, 4:44 a.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36428/
> -----------------------------------------------------------
> 
> (Updated July 12, 2015, 4:44 a.m.)
> 
> 
> Review request for mesos, Ben Mahler, Kapil Arya, and Timothy Chen.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Kapil: this appears to have been introduced in your commit 3a179ce2. Can you 
> please verify this should be removed.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/containerizer.cpp 
> 47d146125dfd4ea909e7ec9d94f41cfa11d035e5 
> 
> Diff: https://reviews.apache.org/r/36428/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ian Downes
> 
>

Reply via email to