> On Dec. 19, 2014, 2:05 a.m., Adam B wrote: > > include/mesos/resources.hpp, lines 64-69 > > <https://reviews.apache.org/r/28698/diff/2/?file=793361#file793361line64> > > > > Does this always assume STATIC? Shouldn't you be able to parse a string > > into name/value/role while also specifying the reservation type?
This one was intentionally put off for 2 reasons. 1. We would have to change the syntax for resources. Current syntax is "name(role):value;...", how should we introduce the dynamic reservation there? perhaps "name[role]:value"? I mentioned this to Tim Chen and he suggested maybe we just ditch this and resort to a JSON format or something more explicit. 2. I don't think we quite want to allow slave resources on the command line to include dynamic reservations. If we allow dynamic reservations to be specified here, we would have to explicitly check for this in the command line and maybe other places we want to disallow it. > On Dec. 19, 2014, 2:05 a.m., Adam B wrote: > > include/mesos/resources.hpp, lines 56-62 > > <https://reviews.apache.org/r/28698/diff/2/?file=793361#file793361line56> > > > > So you expect that this will be called by something (e.g. checkpoint > > recovery) that already knows whether to interpret the strings as a static > > or dynamic reservation? I added it because I thought it may be useful for tests, but we don't actually need it nor use it currently. I can remove it for now. - Michael ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28698/#review65561 ----------------------------------------------------------- On Dec. 30, 2014, 2:44 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28698/ > ----------------------------------------------------------- > > (Updated Dec. 30, 2014, 2:44 a.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, Jie Yu, and > Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Modified Resources to account for reservation type. > > > Diffs > ----- > > include/mesos/resources.hpp f1517b73def9aff4039f95a89f66208ba1d21c0f > src/common/resources.cpp c17e1791130e7d545bb7cdd54d97d65325d3a69e > > Diff: https://reviews.apache.org/r/28698/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >
