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

Haibo Chen commented on YARN-6513:
----------------------------------

Thanks [~wilfreds] for the patch. One nit on the patch, we pass 0 to the 
HashSet constructor given that we  don't reuse  it.

> Fix for FindBugs getPendingLogFilesToUpload() possible NPE
> ----------------------------------------------------------
>
>                 Key: YARN-6513
>                 URL: https://issues.apache.org/jira/browse/YARN-6513
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Wilfred Spiegelenburg
>            Assignee: Wilfred Spiegelenburg
>         Attachments: YARN-6513.01.patch
>
>
> {code}
> Possible null pointer dereference in 
> org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File)
>  due to return value of called method
> Bug type NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE (click for details) 
> In class org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue
> In method 
> org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File)
> Local variable stored in JVM register ?
> Method invoked at AggregatedLogFormat.java:[line 314]
> Known null at AggregatedLogFormat.java:[line 314]
> {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]

Reply via email to