You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Qian Zhang <zh...@gmail.com> on 2019/01/07 00:51:23 UTC

Re: Review Request 69481: Deallocated the shared persistent volume's gid when it is removed.

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

(Updated Jan. 7, 2019, 8:51 a.m.)


Review request for mesos, Andrei Budnik, Gilbert Song, Greg Mann, Ilya Pronin, and Jie Yu.


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


Repository: mesos


Description
-------

Deallocated the shared persistent volume's gid when it is removed.


Diffs (updated)
-----

  src/slave/slave.cpp ad3b693a716cf6103345a157bf28dd60a7b07d32 


Diff: https://reviews.apache.org/r/69481/diff/3/

Changes: https://reviews.apache.org/r/69481/diff/2-3/


Testing
-------


Thanks,

Qian Zhang


Re: Review Request 69481: Deallocated the shared persistent volume's gid when it is removed.

Posted by Gilbert Song <so...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69481/#review213133
-----------------------------------------------------------


Ship it!




Ship It!

- Gilbert Song


On Jan. 6, 2019, 4:51 p.m., Qian Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69481/
> -----------------------------------------------------------
> 
> (Updated Jan. 6, 2019, 4:51 p.m.)
> 
> 
> Review request for mesos, Andrei Budnik, Gilbert Song, Greg Mann, Ilya Pronin, and Jie Yu.
> 
> 
> Bugs: MESOS-8813
>     https://issues.apache.org/jira/browse/MESOS-8813
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Deallocated the shared persistent volume's gid when it is removed.
> 
> 
> Diffs
> -----
> 
>   src/slave/slave.cpp ad3b693a716cf6103345a157bf28dd60a7b07d32 
> 
> 
> Diff: https://reviews.apache.org/r/69481/diff/3/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Qian Zhang
> 
>