Azrael Seoeun updated OOZIE-2644:
    Attachment: OOZIE-2644.1.patch

The patch is addressed comments.

As of now, CallableQueueService increase one more count in queued group for 
instrumentation if CompositeCallable queued. So, if CompositeCallable which 
contain 2 callables is queued, it add 3 to queued count. I am not sure it is 
intended or not. If it is not intended, I can create separate Jira issue.

And this patch fix the NullPointerException when CoordActionNotification 
executed from CoordActionKillXCommand. This is caused the runConf of actionBean 
is not set.  I can create separate Jira issue if needed.

Configuration conf;
        try {
            conf = new XConfiguration(new 
        catch (IOException e1) {
            LOG.warn("Configuration parse error. read from DB :" + 
            throw new CommandException(ErrorCode.E1005, e1.getMessage(), e1);

> Skip queuing Notification Commands when there's nothing to notify
> -----------------------------------------------------------------
>                 Key: OOZIE-2644
>                 URL: https://issues.apache.org/jira/browse/OOZIE-2644
>             Project: Oozie
>          Issue Type: Improvement
>    Affects Versions: 3.1.3
>            Reporter: Robert Kanter
>            Assignee: Azrael Seoeun
>             Fix For: 5.0.0
>         Attachments: OOZIE-2644.1.patch
> When you use the 
> [Workflow|https://oozie.apache.org/docs/4.2.0/WorkflowFunctionalSpec.html#a5_Workflow_Notifications]
>  or 
> [Coordinator|https://oozie.apache.org/docs/4.2.0/CoordinatorFunctionalSpec.html#a15._Coordinator_Notifications]
>  Notification features, Oozie can end up queuing up a lot of 
> {{WorkflowNotificationXCommand}} and {{CoordActionNotificationXCommand}}.  
> This happens even if there's no notification configured on the job (which I 
> imagine is most of the time); in this case, the {{execute}} method simply 
> does nothing.  This is wasteful and clogs the queue up.  
> We should change the code so that it doesn't queue up one of these Commands 
> unless there's actually a URL to notify.

This message was sent by Atlassian JIRA

Reply via email to