----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64551/#review193594 -----------------------------------------------------------
In the commit message: `s/and garbage collect/and garbage collects/` src/resource_provider/storage/provider.cpp Lines 904-905 (patched) <https://reviews.apache.org/r/64551/#comment272162> Why can't we reconstruct them? The list of status returned by the SUM will contain a terminal update, but `terminated` will be `false`. src/resource_provider/storage/provider.cpp Lines 2538 (patched) <https://reviews.apache.org/r/64551/#comment272163> s/refactor/refactoring/ - Gaston Kleiman On Dec. 12, 2017, 9:52 a.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64551/ > ----------------------------------------------------------- > > (Updated Dec. 12, 2017, 9:52 a.m.) > > > Review request for mesos, Gaston Kleiman, Greg Mann, and Jie Yu. > > > Repository: mesos > > > Description > ------- > > This patch uses the status update manager to send status updates and > recover operations that are checkpointed as completed in the status > update manager but still in the pending list in SLRP. It also forwards > the acknowledgements to the status update manager, and garbage collect > the metadata for offer operations. > > > Diffs > ----- > > src/resource_provider/daemon.hpp 7c513a2b7c7ca958f9ea57326493f74df5b67c8f > src/resource_provider/daemon.cpp 7c783e309672a2d93532fb17f13afcb96d5a7006 > src/resource_provider/local.hpp 46111d9f8969ea47e695d08832cb5a9ea60d1971 > src/resource_provider/local.cpp d1d683534d186e77a2beb32c978f2aaea289b086 > src/resource_provider/storage/provider.hpp > 374f837ad693820afffb86bd1e54915f6796d088 > src/resource_provider/storage/provider.cpp > e806f44ef33405d4a2b133576c60be56e9fe3435 > src/slave/paths.hpp d645d871c36bbe8e766a98650f6aa23b6eab65d8 > src/slave/paths.cpp b8004e76964abc210820368a89dbfa6928ef7bfd > > > Diff: https://reviews.apache.org/r/64551/diff/1/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Chun-Hung Hsiao > >
