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

Vinod Kumar Vavilapalli commented on YARN-1569:
-----------------------------------------------

Let's not do this. If we are getting wrong events that are incorrectly getting 
routed, I'd rather catch them at compile time instead of putting out a log that 
nobody will look at. I propose we close this as WONT FIX.

> For handle(SchedulerEvent) in FifoScheduler and CapacityScheduler, 
> SchedulerEvent should get checked (instanceof) for appropriate type before 
> casting
> -----------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-1569
>                 URL: https://issues.apache.org/jira/browse/YARN-1569
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: scheduler
>            Reporter: Junping Du
>            Assignee: zhihai xu
>            Priority: Minor
>              Labels: newbie
>         Attachments: yarn-1569.patch
>
>
> As following: http://wiki.apache.org/hadoop/CodeReviewChecklist, we should 
> always check appropriate type before casting. 
> handle(SchedulerEvent) in FifoScheduler and CapacityScheduler didn't check so 
> far (no bug there now) but should be improved as FairScheduler.



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

Reply via email to