[ 
https://issues.apache.org/jira/browse/OOZIE-2055?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14225339#comment-14225339
 ] 

Rohini Palaniswamy commented on OOZIE-2055:
-------------------------------------------

+1. I believe the change to queue commands instead of call will also make the 
service run faster.

bq. . -1 the patch does not add/modify any testcase
 Existing tests for PauseTransitService should be good enough for this change.

Please change log message "Unable to queue the callables commands for 
CoordMaterializeTriggerRunnable" to  "Unable to queue the callables commands 
for PauseTransitService" before checking in. 

> PauseTransitService does not proceed forward if any job has issue 
> ------------------------------------------------------------------
>
>                 Key: OOZIE-2055
>                 URL: https://issues.apache.org/jira/browse/OOZIE-2055
>             Project: Oozie
>          Issue Type: Bug
>            Reporter: Purshotam Shah
>            Assignee: Purshotam Shah
>         Attachments: OOZIE-2055-V1.patch
>
>
> {code}
>  // start bundles as needed;
>             try {
>                 jobList = jpaService.execute(new 
> BundleJobsGetNeedStartJPAExecutor(d));
>                 if (jobList != null) {
>                     for (BundleJobBean bundleJob : jobList) {
>                         bundleJob.setKickoffTime(d);
>                         new BundleStartXCommand(bundleJob.getId()).call();
>                         LOG.debug("Calling BundleStartXCommand for bundle job 
> = " + bundleJob.getId());
>                     }
>                 }
>             }
>             catch (Exception ex) {
>                 LOG.warn("Exception happened when pausing/unpausing/starting 
> Bundle jobs", ex);
>             }
> {code}
> If any of job has issue, then exception will be thrown and 
> PauseTransitService will ignore all other jobs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to