mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anindya Sinha <>
Subject Re: Review Request 59194: Validate DESTROY operation in `Resources::apply()`.
Date Sat, 13 May 2017 04:50:44 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated May 13, 2017, 4:50 a.m.)

Review request for mesos, James Peach and Jiang Yan Xu.


Addressed review comments.

Bugs: MESOS-7403

Repository: mesos

Description (updated)

Ensure that applying `DESTROY` operation removes this volume. This is
needed to handle `DESTROY` of shared volumes to ensure that the
resultant `Resources` object has this `Resource` removed (i.e. we
allow this operation to succeed if the original `Resources` object
had a single copy of the shared resource).

Diffs (updated)

  src/common/resources.cpp f6f02eb0a7ca05ab8fd9a670f32428862009b7d5 
  src/tests/resources_tests.cpp 4cf320d802a749f1419ac5b9f63b6c73b0c974be 
  src/v1/resources.cpp cad069defb34d5ccc20a0e083eb88cb80f70e415 




All tests passed.


Anindya Sinha

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message