[ https://issues.apache.org/jira/browse/OOZIE-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13753959#comment-13753959 ]
Mona Chitnis commented on OOZIE-1448: ------------------------------------- {quote} Ideally, this piece should be removed and the RecoveryService should have way to recover this command. Also, due to no recovery of this command, its always being called directly instead of being queued. {quote} I am looking at removing this piece as part of DB calls optimization patch OOZIE-1503 > A CoordActionUpdateXCommand gets queued for all workflows even if they were > not launched by a coordinator > --------------------------------------------------------------------------------------------------------- > > Key: OOZIE-1448 > URL: https://issues.apache.org/jira/browse/OOZIE-1448 > Project: Oozie > Issue Type: Bug > Reporter: Robert Kanter > Assignee: Robert Kanter > Attachments: OOZIE-1448.patch, OOZIE-1448.patch, OOZIE-1448.patch, > OOZIE-1448.patch, OOZIE-1448.patch > > > Once a workflow (that wasn't started by a coordinator) ends, there's almost > always a warning/error logged that looks like this: > {noformat} > 2013-07-09 16:16:54,711 WARN CoordActionUpdateXCommand:542 - USER[rkanter] > GROUP[-] TOKEN[] APP[pig-wf] JOB[0000000-130709161625948-oozie-rkan-W] > ACTION[-] E1100: Command precondition does not hold before execution, [, > coord action is null], Error Code: E1100 > {noformat} > The error is harmless, but it tends to confuse users who think that something > went wrong. It also means that we have an extra unnecessary command in the > queue for every workflow that wasn't started by a coordinator. > In SignalXCommand, there is a line like this: > {code:java} > new CoordActionUpdateXCommand(wfJob).call(); //Note: Called even if wf is > not necessarily instantiated by coordinator > {code} > The comment is part of the original code, and makes me think that this was > done on purpose or perhaps when there wasn't a good way to check if a > workflow was started by a coordinator? > I think we can fix this by simply checking if the parent of {{wfJob}} is a > coordinator. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira