[jira] [Updated] (MAPREDUCE-6901) Remove @deprecated tags from DistributedCache

["Jason Lowe (JIRA)" <ji...@apache.org>]

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

Jason Lowe updated MAPREDUCE-6901:
----------------------------------
    Priority: Major  (was: Critical)

Lowering the priority since I don't see this as an impediment to the 3.x release.  If we were removing the DistributedCache API as originally intended then it would be, but that is not currently what this is proposing.

If we are going to remove the Deprecated flags on the methods but not the properties then it would be nice if the non-deprecated methods didn't use/set those deprecated property names.  It's always weird when the source is yelling at users to stop using deprecated things and then turns around and uses it everywhere in non-deprecated functions.  It's not self-consistent and may be one of the reasons many of the downstream projects never took the deprecation warnings on DistributedCache seriously.

Speaking of which, I noticed that we are removing the deprecations from org.apache.hadoop.filecache.DistributedCache but not from org.apache.mapreduce.filecache.DistributedCache, yet Job calls the latter's deprecated methods all over the place.  Seems like that should be cleaned up as part of this as well.  Oddly enough some of the deprecations say to call the Job method instead, but that Job method turns around and calls the deprecated DistributedCache method.  Sigh.

> Remove @deprecated tags from DistributedCache
> ---------------------------------------------
>
>                 Key: MAPREDUCE-6901
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6901
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: distributed-cache
>    Affects Versions: 3.0.0-alpha3
>            Reporter: Ray Chiang
>            Assignee: Ray Chiang
>         Attachments: MAPREDUCE-6901.001.patch
>
>
> Doing this as part of Hadoop 3 cleanup.
> DistributedCache has been marked as deprecated forever to the point where the change that did it isn't in Git.
> I don't really have a preference for whether we remove it or not, but I'd like to have a discussion and have it properly documented as a release not for Hadoop 3 before we hit final release.  At the very least we can have a Release Note that will sum up whatever discussion we have here.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-help@hadoop.apache.org