----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8670/#review14781 -----------------------------------------------------------
branches/hcat-intre/client/src/main/java/org/apache/oozie/cli/OozieCLI.java <https://reviews.apache.org/r/8670/#comment31344> a minor comment...if pushdependencies are null, the instance separator and maskIfNull character will be shown at the end..i think we should skip showing both these characters when pushdeps are null same comment at other place - Virag Kothari On Dec. 20, 2012, 6:18 p.m., Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8670/ > ----------------------------------------------------------- > > (Updated Dec. 20, 2012, 6:18 p.m.) > > > Review request for oozie. > > > Description > ------- > > https://issues.apache.org/jira/browse/OOZIE-1135 > > > This addresses bug OOZIE-1135. > https://issues.apache.org/jira/browse/OOZIE-1135 > > > Diffs > ----- > > branches/hcat-intre/client/src/main/java/org/apache/oozie/cli/OozieCLI.java > 1424233 > > Diff: https://reviews.apache.org/r/8670/diff/ > > > Testing > ------- > > unit test n/a. Test via submitting job and checking job -info -verbose on CLI > > > Thanks, > > Mona Chitnis > >
