> On Dec. 23, 2014, 1:26 a.m., Adam B wrote: > > src/launcher/fetcher.cpp, lines 285-286 > > <https://reviews.apache.org/r/28975/diff/1/?file=789894#file789894line285> > > > > How can fetcherInfo not have a commandinfo if it's required? I would > > think it would have errored in the parse function.
Wasn't sure if we double-check required field. Apparently not. Removed. > On Dec. 23, 2014, 1:26 a.m., Adam B wrote: > > src/launcher/fetcher.cpp, lines 290-291 > > <https://reviews.apache.org/r/28975/diff/1/?file=789894#file789894line290> > > > > Ditto on another required field, but maybe check that it's not empty > > string? Now checking for empty string. > On Dec. 23, 2014, 1:26 a.m., Adam B wrote: > > src/slave/containerizer/fetcher.hpp, lines 47-49 > > <https://reviews.apache.org/r/28975/diff/1/?file=789895#file789895line47> > > > > Are we going to do this sometime soon too? MESOS-2172 should deal with this one way or another. - Bernd ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28975/#review65900 ----------------------------------------------------------- On Dec. 11, 2014, 4:48 p.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28975/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2014, 4:48 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Bugs: MESOS-2173 > https://issues.apache.org/jira/browse/MESOS-2173 > > > Repository: mesos-git > > > Description > ------- > > Changed env var ensemble into one env var containing JSON, parsing this to > protobuf for use. Adapted fetcher tests. > > > Diffs > ----- > > include/mesos/fetcher/fetcher.hpp PRE-CREATION > include/mesos/fetcher/fetcher.proto PRE-CREATION > src/Makefile.am 86161fe7a8bdd86958d24adb74d434cd92d7dfb8 > src/launcher/fetcher.cpp 2f54d9e061d6b56a9b9cd857e5a908fee2847e68 > src/slave/containerizer/fetcher.hpp > 12b81b1065b6b0a0368ebd44326c75c1d4e4ba79 > src/slave/containerizer/fetcher.cpp > d702a9ccc15b21ba26fb9356aa69da15dfd5973e > src/tests/fetcher_tests.cpp 9e48392d31c886bad60abc8999fe293f55f589b0 > > Diff: https://reviews.apache.org/r/28975/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Bernd Mathiske > >
