[ 
https://issues.apache.org/jira/browse/HADOOP-2202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Adrian Woodhead updated HADOOP-2202:
------------------------------------

        Fix Version/s: 0.16.0
             Assignee: Doug Cutting
    Affects Version/s: 0.15.0
               Status: Patch Available  (was: Open)

Fix should be self-explanatory when you look at the code.

As an aside, I always prefer to use the interface (e.g. List) rather than the 
concrete implementation (e.g. ArrayList) wherever possible, especially in 
methods which are in a public API (this is also recommended by Sun and many 
others). I notice the Job class exposes the depending jobs as an ArrayList, I 
think this should be changed to List. If you agree maybe a separate Jira issue 
is in order? I'm quite happy to code up a patch for it.

> JobControl.java generates compiler warnings with Java6
> ------------------------------------------------------
>
>                 Key: HADOOP-2202
>                 URL: https://issues.apache.org/jira/browse/HADOOP-2202
>             Project: Hadoop
>          Issue Type: Bug
>          Components: mapred
>    Affects Versions: 0.15.0
>            Reporter: Doug Cutting
>            Assignee: Doug Cutting
>             Fix For: 0.16.0
>
>         Attachments: HADOOP-2202-1.patch
>
>
> The following compiler warning is seen with Java6:
> {noformat}
>     [javac] 
> /home/cutting/src/hadoop/trunk/src/java/org/apache/hadoop/mapred/jobcontrol/Job.java:221:
>  warning: [unchecked] unchecked call to add(E) as a member of the raw type 
> java.util.ArrayList
>     [javac]       return this.dependingJobs.add(dependingJob);
>     [javac]                                    ^
> {noformat}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to