> On April 17, 2013, 5:53 p.m., Ben Mahler wrote: > > hadoop/TUTORIAL.sh, line 611 > > <https://reviews.apache.org/r/10563/diff/1/?file=281696#file281696line611> > > > > I would argue this is less "portable" as you're using a static path in > > /tmp. > > > > Can you clarify as to why this has anything to do with portability?
You're making the assumption that writing to 'out' is okay. This fails in my testing. /tmp is generally a safe place for temporary files on unix-y systems. - Brenden ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10563/#review19336 ----------------------------------------------------------- On April 16, 2013, 10:08 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10563/ > ----------------------------------------------------------- > > (Updated April 16, 2013, 10:08 p.m.) > > > Review request for mesos. > > > Description > ------- > > Use more portable paths for tutorial test. > > > Diffs > ----- > > hadoop/TUTORIAL.sh f8131cd > > Diff: https://reviews.apache.org/r/10563/diff/ > > > Testing > ------- > > Used in production at Airbnb. > > > Thanks, > > Brenden Matthews > >
