> On Nov. 5, 2015, 1:29 p.m., Bernd Mathiske wrote: > > src/slave/containerizer/fetcher.cpp, line 399 > > <https://reviews.apache.org/r/39595/diff/1/?file=1104942#file1104942line399> > > > > Since this is missing, we are now unconditionally relying on the URI > > being reachable. But why not use the cache entry in case we learn nothing > > new about the URI, because it is unreachable? > > > > This also makes me think that we may want to make checking mtime > > optional?
Good point. I changed the cache reuse policy to now always reuse existing cache entries in case the source URI cannot be queried, so that we do not need to fail anymore just because of a missing source. I left checking of the mtime as is; the implementation already treats mtime as an optional value: since empty `Options` compare equal we would already not take it into account in case it is not available. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39595/#review105232 ----------------------------------------------------------- On Nov. 6, 2015, 12:57 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39595/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2015, 12:57 p.m.) > > > Review request for mesos, Bernd Mathiske, Jan Schlicht, and Till Toenshoff. > > > Bugs: MESOS-3785 > https://issues.apache.org/jira/browse/MESOS-3785 > > > Repository: mesos > > > Description > ------- > > Also added function to query mtime of HDFS resource. > > > Diffs > ----- > > src/hdfs/hdfs.hpp 42c150186f2ce676407e4e00e84bd7e38063d9ba > src/slave/containerizer/fetcher.hpp > c7518a36f6344841880dbb11bfce603fd2791fc0 > src/slave/containerizer/fetcher.cpp > e0d02d5f8f4f6e930a2ae6abe365548af6d1b01f > src/tests/fetcher_cache_tests.cpp 0b3245105b4c1efae54f0bc34f672290819a6f0b > > Diff: https://reviews.apache.org/r/39595/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Bannier > >