Hi folks, Thanks for the great feedback! I addressed almost all of these issues, but had a few things left to wrap up before uploading a new patch -- I'm traveling at the moment, but I should have a new patch by tomorrow. Neil
On Mon, Aug 3, 2015 at 8:45 PM Adam B <a...@mesosphere.io> wrote: > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36867/ > > On July 27th, 2015, 11:36 p.m. PDT, *Adam B* wrote: > > Great first patch. Thanks for updating FrameworkInfo on reregistration with > the master too! > A handful of nits in my first pass. I'll take another look once you've > simplified the tests with Kapil's suggestions. > > On August 3rd, 2015, 1:33 p.m. PDT, *Niklas Nielsen* wrote: > > Any updates here? :) > > Looks like @neilc or somebody resolved some of these issues, but I don't see > a new patch revision with the changes. Neil? > > > - Adam > > On July 27th, 2015, 6:25 p.m. PDT, Neil Conway wrote: > Review request for mesos, Adam B, Connor Doyle, and Niklas Nielsen. > By Neil Conway. > > *Updated July 27, 2015, 6:25 p.m.* > *Bugs: * MESOS-2841 <https://issues.apache.org/jira/browse/MESOS-2841> > *Repository: * mesos > Description > > This is intended to support frameworks that want to offer capabilities to the > rest of the cluster (e.g., storage or some arbitrary third-party service). We > want processes running in the cluster to be able to discover these > capabilities; > however, we don't want to commit to a fixed set of capabilities or how those > capabilities should be represented. Hence, this commit represents this > information using freeform key-value pairs, similar to the labels mechanism > already in use elsewhere. > > Jira: MESOS-2841 > > Testing > > make check > > Diffs > > - include/mesos/mesos.proto (a6748d1cd82238f005c6a49c70d22d095462f1ba) > - src/master/http.cpp (3a1598fad4db03e5f62fd4a6bd26b2bedeee4070) > - src/master/master.hpp (2c924addfb4c52d3048ee6ded13ce638145cc93f) > - src/tests/fault_tolerance_tests.cpp > (7b977f5e8195d9f42b21f36eb36fb156471caa20) > - src/tests/master_tests.cpp (05c148ee1660b86428afe4eda718b17052743a8c) > > View Diff <https://reviews.apache.org/r/36867/diff/> >