> On Jan. 21, 2015, 7:45 p.m., Timothy Chen wrote: > > src/slave/containerizer/fetcher.cpp, line 222 > > <https://reviews.apache.org/r/30006/diff/2/?file=828185#file828185line222> > > > > Can't you directly bind(&collect, pendingDownloads)? > > Bernd Mathiske wrote: > Don't know how to bind to a tempate, and neither does the compiler. Other > ideas?
You can provide the type as part of the bind, if you search for lambda::bind in the code base you'll find an example. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30006/#review68985 ----------------------------------------------------------- On Jan. 22, 2015, 9:38 a.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30006/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2015, 9:38 a.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, and Timothy Chen. > > > Bugs: MESOS-2057 > https://issues.apache.org/jira/browse/MESOS-2057 > > > Repository: mesos-git > > > Description > ------- > > Enables reuse of download results in the cache when multiple fetcher runs > occur concurrently. Optimizes for minimal number of fetcher runs, between 0 > and 2, depending on what is in the cache and what can be obtained from > concurrent runs by simply waiting for their already initiated downloads for > the same content to complete. > > > Diffs > ----- > > src/slave/containerizer/fetcher.hpp > 1db0eaf002c8d0eaf4e0391858e61e0912b35829 > src/slave/containerizer/fetcher.cpp > d290f95251def3952c5ee34f600e1d71467f6293 > > Diff: https://reviews.apache.org/r/30006/diff/ > > > Testing > ------- > > make check, testing whether legacy direct fetching still works, using > existing tests. > > Testing of cache functionality coming in MESOS-2074. > > > Thanks, > > Bernd Mathiske > >
