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

Miklos Szegedi commented on YARN-6372:
--------------------------------------

Thank you for the patch [~yufeigu]. It is not related to the patch, but now 
that you use it, should not the default be set using the predefined constant 
BasicDiskValidator.NAME like below?
{code}
public static final String DEFAULT_DISK_VALIDATOR = BasicDiskValidator.NAME;
{code}

> Add default value for NM disk validator
> ---------------------------------------
>
>                 Key: YARN-6372
>                 URL: https://issues.apache.org/jira/browse/YARN-6372
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>    Affects Versions: 2.7.3, 3.0.0-alpha2
>            Reporter: Yufei Gu
>            Assignee: Yufei Gu
>         Attachments: YARN-6372.001.patch
>
>
> YARN-5137 make DiskChecker pluggable in NodeManager. We should give a default 
> value in case NM does't provide the configuration item.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to