> On Sept. 24, 2015, 9:35 a.m., Alexander Rukletsov wrote: > > include/mesos/master/allocator.hpp, lines 60-61 > > <https://reviews.apache.org/r/37993/diff/10/?file=1077928#file1077928line60> > > > > Not yours, but let's wrap all types and variable names in backticks "`" > > Guangya Liu wrote: > Not quite catch your point, can you please show more detail for what does > this mean?
i.e. s/Try/`Try`/ s/Allocator*/`Allocator*`/ - Joseph ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37993/#review100393 ----------------------------------------------------------- On Sept. 25, 2015, 10:22 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37993/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2015, 10:22 a.m.) > > > Review request for mesos, Adam B, Alexander Rukletsov, and Michael Park. > > > Bugs: MESOS-2224 > https://issues.apache.org/jira/browse/MESOS-2224 > > > Repository: mesos > > > Description > ------- > > Add explanatory comments for Allocator interface > > > Diffs > ----- > > include/mesos/master/allocator.hpp 8100f14220599247a340c919a03f29755b5349d8 > > Diff: https://reviews.apache.org/r/37993/diff/ > > > Testing > ------- > > > Thanks, > > Guangya Liu > >