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

        Hide
        Todd Lipcon added a comment -

        Patch against branch-0.20

        Show
        Todd Lipcon added a comment - Patch against branch-0.20
        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.
        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.
        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.
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development