Junping Du commented on YARN-2940:

bq.  I updated the patch according to you suggestions. After changing the field 
into transient I got another SE_TRANSIENT_FIELD_NOT_RESTORED warning. I fixed 
it by adding a customized readObject method.
Interesting... the conf field should be in exactly the same situation but looks 
like Findbugs didn't complain before. Check some discussions in google, looks 
like it could due to the bug of Findbugs (see 
http://blog.gmane.org/gmane.comp.java.findbugs.general/month=20121101). Anyway, 
the current patch looks good to me. +1. Will commit it soon.

> Fix new findbugs warnings in rest of the hadoop-yarn components
> ---------------------------------------------------------------
>                 Key: YARN-2940
>                 URL: https://issues.apache.org/jira/browse/YARN-2940
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Varun Saxena
>            Assignee: Li Lu
>         Attachments: YARN-2940-121014-1.patch, YARN-2940-121014.patch, 
> YARN-2940-122314.patch, findbugs_report_yarn.zip
> Fix findbugs warnings in the following YARN components:
> hadoop-yarn-applications-distributedshell
> hadoop-yarn-applications-unmanaged-am-launcher
> hadoop-yarn-server-web-proxy
> hadoop-yarn-registry
> hadoop-yarn-server-common
> hadoop-yarn-client

This message was sent by Atlassian JIRA

Reply via email to