ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jun aoki <jun.aoki....@gmail.com>
Subject Re: does test-patch require javadoc examination?
Date Fri, 26 Sep 2014 23:05:17 GMT
Yusaku, thank you for letting me know. I'm going to disable javadoc check
for now in the test-patch.
Let's have an epic and discuss how javadoc should be provided too. The
errors I posted should be cleaned and taken care too before we extend the
test-patch coverage.

On Fri, Sep 26, 2014 at 3:27 PM, Yusaku Sako <yusaku@hortonworks.com> wrote:

> Jun,
>
> No, we have not been building/testing javadoc, but it would be great to
> add.
>
> Yusaku
>
>
> On Fri, Sep 26, 2014 at 3:19 PM, jun aoki <jun.aoki.dev@gmail.com> wrote:
> > Hi Ambari devs,
> >
> > The test-patch job fails currently due to mvn javadoc:javadoc fails.
> >
> >
> https://builds.apache.org/view/A-D/view/Ambari/job/Ambari-trunk-test-patch/45/console
> >
> >
> > More specifically this command fails.
> >
> > {code}
> >
> > mvn clean test javadoc:javadoc -DskipTests -Pdocs -DAmbariPatchProcess
> >
> > {code}
> >
> >
> > and the error is here
> >
> >
> https://builds.apache.org/view/A-D/view/Ambari/job/Ambari-trunk-test-patch/45/artifact/patch-work/trunkJavadocWarnings.txt
> >
> >
> > This is all about a bunch of classic compilation errors.
> >
> > {code}
> >
> > [ERROR] Exit code: 1 -
> >
> /home/jenkins/jenkins-slave/workspace/Ambari-trunk-test-patch/ambari/ambari-server/src/main/java/org/apache/ambari/server/api/MethodOverrideFilter.java:21:
> > error: package org.springframework.http does not exist
> > [ERROR] import org.springframework.http.HttpMethod;
> > [ERROR] ^
> >
> > {code}
> >
> > which, though, can not be observed in the trunk-commit job.
> >
> >
> https://builds.apache.org/view/A-D/view/Ambari/job/Ambari-trunk-Commit/396/
> >
> > The commit job currently does not provide javadoc either, so there
> > should be some class path issue which is different from the regular
> > build.
> >
> > The question is, do we provide javadoc as well as the other rpms? Has
> > anyone tested mvn javadoc:javadoc works? (it fails on my test env as
> > well)
> >
> >
> >
> > --
> > -jun
>
> --
> CONFIDENTIALITY NOTICE
> NOTICE: This message is intended for the use of the individual or entity to
> which it is addressed and may contain information that is confidential,
> privileged and exempt from disclosure under applicable law. If the reader
> of this message is not the intended recipient, you are hereby notified that
> any printing, copying, dissemination, distribution, disclosure or
> forwarding of this communication is strictly prohibited. If you have
> received this communication in error, please contact the sender immediately
> and delete it from your system. Thank You.
>



-- 
-jun

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message