> On March 6, 2015, 3 a.m., Alexander Rukletsov wrote: > > src/common/resources.cpp, line 26 > > <https://reviews.apache.org/r/31664/diff/2/?file=883420#file883420line26> > > > > Why do we need this include? > > Michael Park wrote: > We need the `inline std::size_t hash_value(const SlaveID& slaveId);` from > it for `hashmap<SlaveID, Resources>`.
That was my guess as well. Seems a bit counter-intuitive, but I don't have a better proposal. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31664/#review75452 ----------------------------------------------------------- On March 3, 2015, 8:27 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31664/ > ----------------------------------------------------------- > > (Updated March 3, 2015, 8:27 p.m.) > > > Review request for mesos, Alexander Rukletsov and Ben Mahler. > > > Bugs: MESOS-2373 > https://issues.apache.org/jira/browse/MESOS-2373 > > > Repository: mesos > > > Description > ------- > > Convenience functions to help aggregate `hashmap<SlaveID, Resources>`. Used > in [r31665](https://reviews.apache.org/r/31665). > > > Diffs > ----- > > include/mesos/resources.hpp da6d48871a0061d8bbf5e681dd6e7edac659d812 > src/common/resources.cpp 2c99b6884d7296099e19e2e3182cbe11b5e1e059 > > Diff: https://reviews.apache.org/r/31664/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >
