----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62903/#review188452 -----------------------------------------------------------
Patch looks great! Reviews applied: [63001, 62903] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Oct. 17, 2017, 11:45 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62903/ > ----------------------------------------------------------- > > (Updated Oct. 17, 2017, 11:45 p.m.) > > > Review request for mesos, Benjamin Bannier, Jie Yu, and Jan Schlicht. > > > Bugs: MESOS-8107 > https://issues.apache.org/jira/browse/MESOS-8107 > > > Repository: mesos > > > Description > ------- > > Now a resource provider first `SUBSCRIBE` to the resource provider > manager without resources to get its ID, then use the ID to prepare the > checkpoints for recovery and persistent work directory, and then > update its total resources and pending operations through > `UPDATE_STATE`. > > > Diffs > ----- > > include/mesos/resource_provider/resource_provider.proto > f5a9073075327019fd133bd51265f695ef464845 > include/mesos/v1/resource_provider/resource_provider.proto > e5cbede5b6e57a8641fca1ebfee5454f292cc24c > src/resource_provider/manager.cpp 31fcb789f5ab907511e868c374c49f7457a33ed3 > src/resource_provider/validation.cpp > d2927227f60ab0d4ae2481ad73a31ee444b48ee0 > src/tests/resource_provider_manager_tests.cpp > ca49e1f0203494fc8b4a4507c33e5a3885a14a59 > src/tests/resource_provider_validation_tests.cpp > f182bff4670318e9de22c2915c5dbb423a74ad6c > > > Diff: https://reviews.apache.org/r/62903/diff/7/ > > > Testing > ------- > > make check > > > Thanks, > > Chun-Hung Hsiao > >
