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

Chris Douglas commented on HADOOP-12180:
----------------------------------------

bq. SysInfo#newInstance - if OS type is not Linux or Windows, is that a 
security issue? Would RuntimeException be more appropriate than 
SecurityException?

It throws {{SecurityException}} because {{RCP::getResourceCalculatorPlugin}} 
caught it when creating the impl fails, not for any particularly good reason. 
I'm happy to replace it with something else including {{RuntimeException}}, 
though ideally there's a more specific type to catch at RCP to avoid swallowing 
exceptions there.

> Move ResourceCalculatorPlugin from YARN to Common
> -------------------------------------------------
>
>                 Key: HADOOP-12180
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12180
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: util
>            Reporter: Chris Douglas
>            Assignee: Chris Douglas
>         Attachments: HADOOP-12180.000.patch, HADOOP-12180.001.patch, 
> HADOOP-12180.002.patch, HADOOP-12180.003.patch
>
>
> Some of the monitoring functions could be moved from YARN to Common for 
> easier sharing



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to