----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13536/#review25110 -----------------------------------------------------------
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestFsActionExecutor.java <https://reviews.apache.org/r/13536/#comment49334> This testcase does not help. Even the current code will pass with this testcase without the fix in this patch. Please add testcase that exercises resolveToFullPath through chmod or chgrp given a namenodePath instead of just absolute path. - Rohini Palaniswamy On Aug. 13, 2013, 11:40 p.m., Ryota Egashira wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13536/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2013, 11:40 p.m.) > > > Review request for oozie. > > > Bugs: oozie-1495 > https://issues.apache.org/jira/browse/oozie-1495 > > > Repository: oozie > > > Description > ------- > > https://issues.apache.org/jira/browse/OOZIE-1495 > > > Diffs > ----- > > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java > 1513573 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestFsActionExecutor.java > 1513573 > > Diff: https://reviews.apache.org/r/13536/diff/ > > > Testing > ------- > > > Thanks, > > Ryota Egashira > >
