Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-1734

Un-deprecate the old MapReduce API in the 0.20 branch

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.20.3
    • Component/s: documentation
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      This issue is to un-deprecate the "old" MapReduce API (in o.a.h.mapred) in the next 0.20 release, as discussed at http://www.mail-archive.com/mapreduce-dev@hadoop.apache.org/msg01833.html

        Activity

        Tom White created issue -
        Hide
        Todd Lipcon added a comment -

        Patch against branch-0.20

        Show
        Todd Lipcon added a comment - Patch against branch-0.20
        Todd Lipcon made changes -
        Field Original Value New Value
        Attachment mapreduce-1734.txt [ 12466843 ]
        Todd Lipcon made changes -
        Assignee Todd Lipcon [ tlipcon ]
        Show
        Todd Lipcon added a comment - https://reviews.apache.org/r/209/
        Hide
        Tom White added a comment -

        +1. I generated javadoc for this change and it looked good.

        Show
        Tom White added a comment - +1. I generated javadoc for this change and it looked good.
        Hide
        Todd Lipcon added a comment -

        Thanks Tom. Committed to branch-0.20.

        Show
        Todd Lipcon added a comment - Thanks Tom. Committed to branch-0.20.
        Todd Lipcon made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Hadoop Flags [Reviewed]
        Resolution Fixed [ 1 ]
        Hide
        Owen O'Malley added a comment -

        I already made the tag for the 0.20.3. Pushing this to 0.20.4.

        Show
        Owen O'Malley added a comment - I already made the tag for the 0.20.3. Pushing this to 0.20.4.
        Owen O'Malley made changes -
        Fix Version/s 0.20.4 [ 12316034 ]
        Fix Version/s 0.20.3 [ 12314813 ]
        Owen O'Malley made changes -
        Fix Version/s 0.20.3 [ 12314813 ]
        Fix Version/s 0.20.4 [ 12316034 ]
        Hide
        Harsh J added a comment -

        This was a decision, so it should also go into one of the current 0.20.2xx release. Helps avoid a lot of confusion.

        Show
        Harsh J added a comment - This was a decision, so it should also go into one of the current 0.20.2xx release. Helps avoid a lot of confusion.
        Hide
        Arun C Murthy added a comment -

        Harsh - good point. Do you want to provide a patch asap? I'll commit to 0.20.2xx, thanks!

        Show
        Arun C Murthy added a comment - Harsh - good point. Do you want to provide a patch asap? I'll commit to 0.20.2xx, thanks!
        Hide
        Arun C Murthy added a comment -

        I checked, the original still applies clean - should I go ahead commit it?

        Show
        Arun C Murthy added a comment - I checked, the original still applies clean - should I go ahead commit it?
        Hide
        Arun C Murthy added a comment -

        Re-open to commit to 0.20.205.

        Show
        Arun C Murthy added a comment - Re-open to commit to 0.20.205.
        Arun C Murthy made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Matt Foley added a comment -

        +1. I agree with Arun and Harsh. Please commit to 205 and 20-security. Thanks.

        Show
        Matt Foley added a comment - +1. I agree with Arun and Harsh. Please commit to 205 and 20-security. Thanks.
        Hide
        Harsh J added a comment -

        Thanks Arun, for taking this up. Sorry I hadn't hit 'watch' on this one when I got here last time, so did not get your request in time. Would be glad to add in anything else you'd want me to.

        Show
        Harsh J added a comment - Thanks Arun, for taking this up. Sorry I hadn't hit 'watch' on this one when I got here last time, so did not get your request in time. Would be glad to add in anything else you'd want me to.
        Hide
        Arun C Murthy added a comment -

        Thanks Harsh & Matt. I've committed this to 0.20.205.

        Show
        Arun C Murthy added a comment - Thanks Harsh & Matt. I've committed this to 0.20.205.
        Arun C Murthy made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        252d 17h 38m 1 Todd Lipcon 05/Jan/11 22:38
        Resolved Resolved Reopened Reopened
        249d 22h 32m 1 Arun C Murthy 12/Sep/11 22:11
        Reopened Reopened Resolved Resolved
        25m 42s 1 Arun C Murthy 12/Sep/11 22:37

          People

          • Assignee:
            Todd Lipcon
            Reporter:
            Tom White
          • Votes:
            1 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development