hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jason Lowe (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (MAPREDUCE-6901) Remove @deprecated tags from DistributedCache
Date Fri, 04 Aug 2017 16:00:03 GMT

     [ 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

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

View raw message