Hadoop Common
  1. Hadoop Common
  2. HADOOP-10110

hadoop-auth has a build break due to missing dependency

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.6-alpha, 2.2.0
    • Fix Version/s: 3.0.0, 0.23.11, 2.3.0
    • Component/s: build
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      We have a build break in hadoop-auth if build with maven cache cleaned. The error looks like the follows. The problem exists on both Windows and Linux. If you have old jetty jars in your maven cache, you won't see the error.

      [INFO] ------------------------------------------------------------------------
      [INFO] BUILD FAILURE
      [INFO] ------------------------------------------------------------------------
      [INFO] Total time: 1:29.469s
      [INFO] Finished at: Mon Nov 18 12:30:36 PST 2013
      [INFO] Final Memory: 37M/120M
      [INFO] ------------------------------------------------------------------------
      [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.5.1:testCompile (default-testCompile) on project hadoop-auth: Compilation failure: Compilation failure:
      [ERROR] /home/chuan/trunk/hadoop-common-project/hadoop-auth/src/test/java/org/apache/hadoop/security/authentication/client/AuthenticatorTestCase.java:[84,13] cannot access org.mortbay.component.AbstractLifeCycle
      [ERROR] class file for org.mortbay.component.AbstractLifeCycle not found
      [ERROR] server = new Server(0);
      [ERROR] /home/chuan/trunk/hadoop-common-project/hadoop-auth/src/test/java/org/apache/hadoop/security/authentication/client/AuthenticatorTestCase.java:[94,29] cannot access org.mortbay.component.LifeCycle
      [ERROR] class file for org.mortbay.component.LifeCycle not found
      [ERROR] server.getConnectors()[0].setHost(host);
      [ERROR] /home/chuan/trunk/hadoop-common-project/hadoop-auth/src/test/java/org/apache/hadoop/security/authentication/client/AuthenticatorTestCase.java:[96,10] cannot find symbol
      [ERROR] symbol  : method start()
      [ERROR] location: class org.mortbay.jetty.Server
      [ERROR] /home/chuan/trunk/hadoop-common-project/hadoop-auth/src/test/java/org/apache/hadoop/security/authentication/client/AuthenticatorTestCase.java:[102,12] cannot find symbol
      [ERROR] symbol  : method stop()
      [ERROR] location: class org.mortbay.jetty.Server
      [ERROR] -> [Help 1]
      [ERROR]
      [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
      [ERROR] Re-run Maven using the -X switch to enable full debug logging.
      [ERROR]
      [ERROR] For more information about the errors and possible solutions, please read the following articles:
      [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
      [ERROR]
      [ERROR] After correcting the problems, you can resume the build with the command
      [ERROR]   mvn <goals> -rf :hadoop-auth
      
      

        Issue Links

          Activity

          Hide
          Jason Lowe added a comment -

          Thanks Chuan Liu for the contribution and everyone for the reviews. I committed this to branch-0.23 as well.

          Show
          Jason Lowe added a comment - Thanks Chuan Liu for the contribution and everyone for the reviews. I committed this to branch-0.23 as well.
          Hide
          Konstantin Boudnik added a comment -

          Committed to trunk, branch-2, and branch-2.3. Thank you, Chuan!

          Show
          Konstantin Boudnik added a comment - Committed to trunk, branch-2, and branch-2.3. Thank you, Chuan!
          Hide
          Konstantin Boudnik added a comment -

          I will commit this as soon as my SVN password issue is resolved (hopefully by tomorrow). If someone want to commit it before - please do!

          Show
          Konstantin Boudnik added a comment - I will commit this as soon as my SVN password issue is resolved (hopefully by tomorrow). If someone want to commit it before - please do!
          Hide
          Sangjin Lee added a comment -

          +1 on this patch as well. We have also encountered this issue.

          Show
          Sangjin Lee added a comment - +1 on this patch as well. We have also encountered this issue.
          Hide
          Konstantin Boudnik added a comment -

          +1 on the patch.

          Show
          Konstantin Boudnik added a comment - +1 on the patch.
          Hide
          inza9hi added a comment -

          I downloaded the source code:
          http://mirrors.cnnic.cn/apache/hadoop/common/hadoop-2.2.0/hadoop-2.2.0-src.tar.gz
          And I met the same problem.

          With this patch, I solved the problem.
          It seems the source code have not updated.

          Show
          inza9hi added a comment - I downloaded the source code: http://mirrors.cnnic.cn/apache/hadoop/common/hadoop-2.2.0/hadoop-2.2.0-src.tar.gz And I met the same problem. With this patch, I solved the problem. It seems the source code have not updated.
          Hide
          Arpit Agarwal added a comment -

          What's the target branch for 2.0.x? Thanks.

          Show
          Arpit Agarwal added a comment - What's the target branch for 2.0.x? Thanks.
          Hide
          Konstantin Boudnik added a comment -

          Needed to be ported back into 2.0.x line as it's breaking the build there as well

          Show
          Konstantin Boudnik added a comment - Needed to be ported back into 2.0.x line as it's breaking the build there as well
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk #1587 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1587/)
          HADOOP-10110. hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #1587 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1587/ ) HADOOP-10110 . hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190 ) /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #1613 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1613/)
          HADOOP-10110. hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #1613 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1613/ ) HADOOP-10110 . hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190 ) /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Yarn-trunk #396 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/396/)
          HADOOP-10110. hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #396 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/396/ ) HADOOP-10110 . hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190 ) /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-trunk-Commit #4753 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4753/)
          HADOOP-10110. hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190)

          • /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #4753 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4753/ ) HADOOP-10110 . hadoop-auth has a build break due to missing dependency. (Contributed by Chuan Liu) (arp: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1543190 ) /hadoop/common/trunk/hadoop-common-project/hadoop-auth/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          Hide
          Arpit Agarwal added a comment -

          Also thanks to Colin, Haohui and Brandon Li for re-verifying since we don't have a Jenkins +1.

          Show
          Arpit Agarwal added a comment - Also thanks to Colin, Haohui and Brandon Li for re-verifying since we don't have a Jenkins +1.
          Hide
          Arpit Agarwal added a comment -

          Committed this to trunk, branch-2 and branch-2.2.

          Thanks for contribution Chuan!

          Show
          Arpit Agarwal added a comment - Committed this to trunk, branch-2 and branch-2.2. Thanks for contribution Chuan!
          Hide
          Arpit Agarwal added a comment -

          +1 for the patch, just verified it.

          I will commit it shortly.

          Show
          Arpit Agarwal added a comment - +1 for the patch, just verified it. I will commit it shortly.
          Hide
          Haohui Mai added a comment -

          I've verified that the patch works as well. It seems that right now jenkins will fail to compile any new patches without this patch, thus it would be great if this patch can land in trunk as soon as possible.

          Show
          Haohui Mai added a comment - I've verified that the patch works as well. It seems that right now jenkins will fail to compile any new patches without this patch, thus it would be great if this patch can land in trunk as soon as possible.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12614482/HADOOP-10110.patch
          against trunk revision .

          -1 patch. Trunk compilation may be broken.

          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/3292//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12614482/HADOOP-10110.patch against trunk revision . -1 patch . Trunk compilation may be broken. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/3292//console This message is automatically generated.
          Hide
          Colin Patrick McCabe added a comment -

          +1. I verified that I could build without the jetty jars in my ~/.m2 directory with this patch.

          Show
          Colin Patrick McCabe added a comment - +1. I verified that I could build without the jetty jars in my ~/.m2 directory with this patch.
          Hide
          Chuan Liu added a comment -

          Attaching a patch that adds the missing dependency. I verified the build on both Linux and Windows.

          Show
          Chuan Liu added a comment - Attaching a patch that adds the missing dependency. I verified the build on both Linux and Windows.

            People

            • Assignee:
              Chuan Liu
              Reporter:
              Chuan Liu
            • Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development