[
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)