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

correct example code in /api/org/apache/hadoop/util/Tool.html

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Duplicate
    • Affects Version/s: 0.20.2
    • Fix Version/s: None
    • Component/s: documentation
    • Labels:
      None
    • Environment:

      None

      Description

      a trivial mistake in example code. But I wondered where Sort() was come from for a few days.

      < int res = ToolRunner.run(new Configuration(), new Sort(), args);

      > int res = ToolRunner.run(new Configuration(), new MyApp(), args);

      1. Tool.html.patch
        0.8 kB
        minoru nishikubo
      2. Tool.java.patch
        0.8 kB
        minoru nishikubo

        Activity

        Hide
        Todd Lipcon added a comment -

        Thanks for noticing this. Would you like to provide a patch against trunk?

        Show
        Todd Lipcon added a comment - Thanks for noticing this. Would you like to provide a patch against trunk?
        Hide
        minoru nishikubo added a comment -

        Tool.java.patch is for src/core/apache/hadoop/util/Tool.java.
        Tool.html.patch may not be necessary, because it will be generated by JavaDoc.

        Show
        minoru nishikubo added a comment - Tool.java.patch is for src/core/apache/hadoop/util/Tool.java. Tool.html.patch may not be necessary, because it will be generated by JavaDoc.
        Hide
        Todd Lipcon added a comment -

        Hi Minoru. First, thanks for helping to contribute! Would you mind uploading a single patch file taken from the root of the SVN checkout (eg using svn diff)? Please see the "Creating a patch" section in http://wiki.apache.org/hadoop/HowToContribute if you need a pointer.

        Show
        Todd Lipcon added a comment - Hi Minoru. First, thanks for helping to contribute! Would you mind uploading a single patch file taken from the root of the SVN checkout (eg using svn diff)? Please see the "Creating a patch" section in http://wiki.apache.org/hadoop/HowToContribute if you need a pointer.
        Hide
        minoru nishikubo added a comment -

        > Hi Minoru. First, thanks for helping to contribute! Would you mind uploading a single patch file taken from the root of the SVN checkout (eg using svn diff)? Please see the "Creating a patch" section in http://wiki.apache.org/hadoop/HowToContribute if you need a pointer.

        Hello Todd! Thank you for your kindness!!
        I setup subversion and checked out the source from trunk, someone (perhaps you?) already fixed the source.

        Show
        minoru nishikubo added a comment - > Hi Minoru. First, thanks for helping to contribute! Would you mind uploading a single patch file taken from the root of the SVN checkout (eg using svn diff)? Please see the "Creating a patch" section in http://wiki.apache.org/hadoop/HowToContribute if you need a pointer. Hello Todd! Thank you for your kindness!! I setup subversion and checked out the source from trunk, someone (perhaps you?) already fixed the source.
        Hide
        Todd Lipcon added a comment -

        Yes, it looks like this was fixed in HADOOP-6504. Resolving as dup.

        Show
        Todd Lipcon added a comment - Yes, it looks like this was fixed in HADOOP-6504 . Resolving as dup.

          People

          • Assignee:
            Unassigned
            Reporter:
            minoru nishikubo
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development