Good catch! I quickly verified that changing to XmlUtils.getSLAElement(action) does not cause any regression in a few related unit tests.
Thanks, Mona On 12/5/13 2:29 PM, "bowen zhang" <[email protected]> wrote: >Hi, >I have a question for people from yahoo who did the SLA work. In >verifySlaElements function in submitXCommand, within the "for" loop, >shouldn't we call XmlUtils.getSLAElement(action) instead of >XmlUtils.getSLAElement(eWfJob) to extract the sla element embedded inside >the action tag? If this is indeed a bug, I am going to file one. >Bowen
