> On July 6, 2015, 5:25 a.m., Adam B wrote: > > docs/attributes-resources.md, line 31 > > <https://reviews.apache.org/r/36173/diff/1/?file=999015#file999015line31> > > > > No sets? Or are attributes implicitly sets?
I was suprised when I read the source code too, but we actually explicitly don't support sets for attributes yet. > On July 6, 2015, 5:25 a.m., Adam B wrote: > > docs/attributes-resources.md, line 39 > > <https://reviews.apache.org/r/36173/diff/1/?file=999015#file999015line39> > > > > 3 types? What about plain text? Plain text is not a supported resource type. Not sure why but that's what the parse code looks like. - Timothy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36173/#review90437 ----------------------------------------------------------- On July 3, 2015, 9:30 p.m., Timothy Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36173/ > ----------------------------------------------------------- > > (Updated July 3, 2015, 9:30 p.m.) > > > Review request for mesos, Adam B, Benjamin Hindman, Bernd Mathiske, and Till > Toenshoff. > > > Repository: mesos > > > Description > ------- > > Update attributes doc to reflect current supported attributes types. > > > Diffs > ----- > > docs/attributes-resources.md 0ae8b5908fe0b3a3499e6d813afbb328a13bdcde > > Diff: https://reviews.apache.org/r/36173/diff/ > > > Testing > ------- > > make > > > Thanks, > > Timothy Chen > >
