Hi, The patch for this should be pretty easy, but does it mean that you decided not to change the API in order to keep it compatible? I like Doug's idea about depreciation of original methods (those exposing ArrayList into public API) and introducing equivalent methods with List interface.
Regards, Lukas On Wed, Jul 9, 2008 at 9:08 AM, Arun C Murthy <[EMAIL PROTECTED]> wrote: > > On Jul 8, 2008, at 3:11 PM, Lukas Vlcek wrote: > > Hi, >> >> Why the Job API is using ArrayList instead of List? >> >> It is used in constructor: >> public Job(JobConf jobConf, ArrayList<Job> dependingJobs) >> >> and one public method: >> public ArrayList<Job> getDependingJobs() >> >> Is there any reason why it is referencing to implementation while it could >> be replaced by interface? >> >> > Originally an oversight, we did file > http://issues.apache.org/jira/browse/HADOOP-2268 > but didn't change it to keep it compatible... > > Arun > -- http://blog.lukas-vlcek.com/
