> On April 12, 2016, 6:49 p.m., Boris Shkolnik wrote: > > docs/learn/documentation/versioned/yarn/yarn-host-affinity.md, line 106 > > <https://reviews.apache.org/r/45601/diff/1/?file=1322576#file1322576line106> > > > > Please expand 'mapping'. It is not clear from the context without > > reading the SAMZA-886. > > Jake Maes wrote: > I don't want to duplicate the hadoop documentation that is linked in the > first sentence, so can you please specify what is unclear? > > Would it help if I said "DNSToSwitchMapping"?
I think the gap is due to the fact that we don't have any documentation on tuning Rack Awarness for Yarn in the Samza website. So, those who are new to Samza will have a tough time digging out the information from the Yarn documentation. 1. You can replace "mapping" with "DNSToSwitchMapping" and also, explain what info this mapping maintains, what will happen if the mapping is not configured etc. 2. The documentation should also clearly indicate whether RackAwarness is a pre-requisite for correct behavior of host-affinity. Can you please incorporate the above points? Thanks! - Navina ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45601/#review128501 ----------------------------------------------------------- On April 1, 2016, 7:49 p.m., Jake Maes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45601/ > ----------------------------------------------------------- > > (Updated April 1, 2016, 7:49 p.m.) > > > Review request for samza, Navina Ramesh, Jagadish Venkatraman, and Yi Pan > (Data Infrastructure). > > > Repository: samza > > > Description > ------- > > SAMZA-886 document rack awareness impact on host affinity > > > Diffs > ----- > > docs/learn/documentation/versioned/yarn/yarn-host-affinity.md > 1d9c29e7ee7fd19fabba55111b5088cc5d36f1fe > > Diff: https://reviews.apache.org/r/45601/diff/ > > > Testing > ------- > > > Thanks, > > Jake Maes > >
