> On Dec. 22, 2017, 5:01 p.m., Attila Sasvari wrote: > > core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java > > Lines 431 (patched) > > <https://reviews.apache.org/r/64808/diff/1/?file=1926502#file1926502line431> > > > > It depends on the actual filesystem the fs action run agains. So it is > > not necessary moved to a trash directory. In HDFS, trash can be disabled.
Thanks. I deleted this line because it could lead to confusions. - Kinga ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64808/#review194448 ----------------------------------------------------------- On Dec. 27, 2017, 9:12 a.m., Kinga Marton wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64808/ > ----------------------------------------------------------- > > (Updated Dec. 27, 2017, 9:12 a.m.) > > > Review request for oozie and Attila Sasvari. > > > Repository: oozie-git > > > Description > ------- > > Decision, ForkJoin, Email and FS ActionExecutors are producing no or very > little logs on info level. This makes it hard to gather usage information and > help with troubleshooting in the case of any issues. Please improve logging > in these classes. > Several subtasks can be separated out from this one > > > Diffs > ----- > > > core/src/main/java/org/apache/oozie/action/control/ControlNodeActionExecutor.java > cc1b6108 > > core/src/main/java/org/apache/oozie/action/decision/DecisionActionExecutor.java > 8c235bda > core/src/main/java/org/apache/oozie/action/email/EmailActionExecutor.java > d59f1d76 > core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java > 63f6104d > > core/src/main/java/org/apache/oozie/action/oozie/SubWorkflowActionExecutor.java > d62cf686 > core/src/main/java/org/apache/oozie/action/ssh/SshActionExecutor.java > 5890b8c1 > > > Diff: https://reviews.apache.org/r/64808/diff/2/ > > > Testing > ------- > > > Thanks, > > Kinga Marton > >
