[
https://issues.apache.org/jira/browse/YARN-6512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15980480#comment-15980480
]
Naganarasimha G R commented on YARN-6512:
-----------------------------------------
Thanks for working on the patch [~wilfreds], In most of the cases procfsDir
i.e. *"/proc/"* will not be empty unless wrong folder is configured using the
constructor, which is a rare case (and not as per the existing flow). But no
harm in having this fixed as per the patch, hence i am ok in getting this
committed. But i feel we can merge 6513 also in this itself to avoid cross
failures in the jenkins result ?
> Fix for FindBugs getProcessList() possible NPE
> ----------------------------------------------
>
> Key: YARN-6512
> URL: https://issues.apache.org/jira/browse/YARN-6512
> Project: Hadoop YARN
> Issue Type: Bug
> Reporter: Wilfred Spiegelenburg
> Assignee: Wilfred Spiegelenburg
> Attachments: YARN-6512.01.patch
>
>
> Findbugs output:
> {code}
> Possible null pointer dereference in
> org.apache.hadoop.yarn.util.ProcfsBasedProcessTree.getProcessList() due to
> return value of called method
> Bug type NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE
> In class org.apache.hadoop.yarn.util.ProcfsBasedProcessTree
> In method org.apache.hadoop.yarn.util.ProcfsBasedProcessTree.getProcessList()
> Value loaded from processDirs
> Dereferenced at ProcfsBasedProcessTree.java:[line 487]
> Known null at ProcfsBasedProcessTree.java:[line 484]
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
