Bowen,
JIRA has explanation. Please update JIRA if you see any issue with
approach.

>Why is it a good idea to throw an exception if one of the coord jobs is
>in "killed" state? In the BundleJobChangeXCommand, the code doesn't even
>attempt to change the coord job. Shouldn't oozie be intelligent >enough
>to do a no-op on a killed coord job?



To let user know the list of coord jobs for which change is not applied.


Puru.

On 9/18/14, 2:11 PM, "bowen zhang" <bowenzhang...@yahoo.com.INVALID> wrote:

>Hi Purshotam,
>Why is it a good idea to throw an exception if one of the coord jobs is
>in "killed" state? In the BundleJobChangeXCommand, the code doesn't even
>attempt to change the coord job. Shouldn't oozie be intelligent enough to
>do a no-op on a killed coord job?
>Bowen
>
>
>
>________________________________
> From: Purshotam Shah <purus...@yahoo-inc.com.INVALID>
>To: "dev@oozie.apache.org" <dev@oozie.apache.org>; Mona Chitnis
><mona.chit...@yahoo.in>; bowen zhang <bowenzhang...@yahoo.com>
>Sent: Wednesday, September 17, 2014 6:17 PM
>Subject: Re: issue after OOZIE-1807
> 
>
>Hi Bowen,
>   BundleJobChangeXCommand command will get applied to bundle and coord
>jobs. It will aggregate message for all killed coord jobs and throw them
>as exception.
>It is similar to chmod command.
>
>JIRA has more details. Let me know if you need any other information.
>
>Puru.
>
>
>
>
>
>On 9/17/14, 6:05 PM, "Mona Chitnis" <mona.chit...@yahoo.in> wrote:
>
>>
>>Puru,
>>Bowen just gave me a call regarding this issue. Can you answer his
>>question? That'll be faster than me digging through the code.
>> Mona Chitnis
>>Yahoo! 
>>
>>     On Wednesday, September 17, 2014 5:51 PM, bowen zhang
>><bowenzhang...@yahoo.com.INVALID> wrote:
>>  
>>
>> Hi guys, 
>>
>>Purshatom, I see you checked oozie-1807 into the trunk. So, I have a
>>question, why does it need to throw an exception when someone wants to
>>change a bundle job where one of its coord job is in KILLED state? Due to
>>the change in BundleJobChangeXCommand, this is throwing exceptions when
>>trying to change a RUNNING bundle job where some of the coord jobs are
>>intentionally killed by the user.
>>Thanks,
>>Bowen
>>
>>
>>  

Reply via email to