Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-3338

trunk doesn't compile after HADOOP-544 was committed

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 0.18.0
    • 0.18.0
    • None
    • None

    Description

      For all the discussion about deprectatoin, etc, the patch for HADOOP-544 has broken trunk! It no longer compiles on Hudson. The contrib eclipse plugin relied on an API that you considered non-Public (JobStatus and perhaps others). Given we have no well defined public API yet, I don't think you can assume that classes declared "public" aren't actually public. At a minimum, the release notes should indicated that methods were removed and not deprecated.

      Build failure:
      http://hudson.zones.apache.org/hudson/view/Hadoop/job/Hadoop-trunk/476/console

      compile:
      [echo] contrib: eclipse-plugin
      [javac] Compiling 45 source files to /zonestorage/hudson/home/hudson/hudson/jobs/Hadoop-trunk/workspace/trunk/build/contrib/eclipse-plugin/classes
      [javac] /zonestorage/hudson/home/hudson/hudson/jobs/Hadoop-trunk/workspace/trunk/src/contrib/eclipse-plugin/src/java/org/apache/hadoop/eclipse/server/HadoopServer.java:119: cannot find symbol
      [javac] symbol : method getJobId()
      [javac] location: class org.apache.hadoop.mapred.JobStatus
      [javac] String jobId = status.getJobId();
      [javac] ^
      [javac] Note: Some input files use or override a deprecated API.
      [javac] Note: Recompile with -Xlint:deprecation for details.
      [javac] Note: Some input files use unchecked or unsafe operations.
      [javac] Note: Recompile with -Xlint:unchecked for details.
      [javac] 1 error

      Can we get this fixed today?

      Attachments

        1. 3338-20080502a.patch
          6 kB
          Christophe Taton

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            kryzthov Christophe Taton
            nidaley Nigel Daley
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment