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

Rohini Palaniswamy commented on OOZIE-1709:
-------------------------------------------

Shwetha,
   Can you put this up in review board please? Since it changes core logic 
don't want to miss anything during review because the patch does not show 
surrounding code. 

> CoordELFunctions.getCurrentInstance() is expensive
> --------------------------------------------------
>
>                 Key: OOZIE-1709
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1709
>             Project: Oozie
>          Issue Type: Bug
>            Reporter: Shwetha G S
>            Assignee: Shwetha G S
>             Fix For: trunk
>
>         Attachments: OOZIE-1709.patch
>
>
> {noformat}
>       while (current.compareTo(calEffectiveTime) <= 0) {
>             current = (Calendar) origCurrent.clone();
>             instanceCount[0]++;
>             current.add(dsTimeUnit.getCalendarUnit(), instanceCount[0] * 
> dsFreq);
>         }
> {noformat}
> This computes current instance wrt coord start and this while loop is 
> expensive for small frequency coords with start time in very past



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to