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

Make checkstyle runnable from root project

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.7.0
    • Fix Version/s: 2.8.0, 3.0.0-alpha1
    • Component/s: build, test
    • Labels:
      None
    • Target Version/s:

      Description

      Checkstyle -1 are hard to debug without submitting to Jenkins and downloading build artifacts.

      With this JIRA we

      1. remove identical checkstyle rules in hdfs which is identical to common
      2. move checkstyle plugin to the parent pom
      3. update rules to ignore import
      1. HADOOP-11889.001.patch
        12 kB
        Gera Shegalov
      2. HADOOP-11889.002.patch
        20 kB
        Gera Shegalov
      3. HADOOP-11889.003.patch
        21 kB
        Gera Shegalov
      4. HADOOP-11889.004.patch
        22 kB
        Gera Shegalov
      5. HADOOP-11889.004.patch
        22 kB
        Gera Shegalov

        Issue Links

          Activity

          Hide
          jira.shegalov Gera Shegalov added a comment -

          001 patch

          Show
          jira.shegalov Gera Shegalov added a comment - 001 patch
          Hide
          cnauroth Chris Nauroth added a comment -

          Hi Gera Shegalov. Do you think it makes sense to combine this with HADOOP-11778, which is upgrading the checkstyle version? There is a patch attached there, but it's not ready to commit, pending some code review feedback I gave.

          Show
          cnauroth Chris Nauroth added a comment - Hi Gera Shegalov . Do you think it makes sense to combine this with HADOOP-11778 , which is upgrading the checkstyle version? There is a patch attached there, but it's not ready to commit, pending some code review feedback I gave.
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Hi Chris Nauroth, sure, I missed HADOOP-11778. I can add version upgrades to this patch.

          Show
          jira.shegalov Gera Shegalov added a comment - Hi Chris Nauroth , sure, I missed HADOOP-11778 . I can add version upgrades to this patch.
          Hide
          aw Allen Wittenauer added a comment -

          Will checkstyle:checkstyle still work from individual modules? HADOOP-11866 changes the checkstyle test-patch.sh support from global report to a per-module with (naive) filtering.

          Show
          aw Allen Wittenauer added a comment - Will checkstyle:checkstyle still work from individual modules? HADOOP-11866 changes the checkstyle test-patch.sh support from global report to a per-module with (naive) filtering.
          Hide
          aw Allen Wittenauer added a comment -

          (and yes, I'm currently working on making that filtering smarter)

          Show
          aw Allen Wittenauer added a comment - (and yes, I'm currently working on making that filtering smarter)
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 32s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 28s There were no new javac warning messages.
          +1 javadoc 9m 35s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          +1 install 1m 32s mvn install still works.
          +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
          +1 common tests 23m 13s Tests passed in hadoop-common.
          +1 tools/hadoop tests 6m 11s Tests passed in hadoop-distcp.
          +1 hdfs tests 172m 43s Tests passed in hadoop-hdfs.
              236m 11s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729198/HADOOP-11889.001.patch
          Optional Tests javadoc javac unit
          git revision trunk / 8f82970
          hadoop-common test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/artifact/patchprocess/testrun_hadoop-common.txt
          hadoop-distcp test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/artifact/patchprocess/testrun_hadoop-distcp.txt
          hadoop-hdfs test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/artifact/patchprocess/testrun_hadoop-hdfs.txt
          Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/testReport/
          Java 1.7.0_55
          uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 32s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 28s There were no new javac warning messages. +1 javadoc 9m 35s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 install 1m 32s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 common tests 23m 13s Tests passed in hadoop-common. +1 tools/hadoop tests 6m 11s Tests passed in hadoop-distcp. +1 hdfs tests 172m 43s Tests passed in hadoop-hdfs.     236m 11s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729198/HADOOP-11889.001.patch Optional Tests javadoc javac unit git revision trunk / 8f82970 hadoop-common test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/artifact/patchprocess/testrun_hadoop-common.txt hadoop-distcp test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/artifact/patchprocess/testrun_hadoop-distcp.txt hadoop-hdfs test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/artifact/patchprocess/testrun_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/testReport/ Java 1.7.0_55 uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6215/console This message was automatically generated.
          Hide
          jeagles Jonathan Eagles added a comment -

          I hate to comment on this late. Can we please follow the multi-model example method documented by the checkstyle maven plugin. This will allow us to retain the one definition of checkstyle plugins as well as make it play nicely with eclipse.

          http://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html
          https://jgonian.wordpress.com/2010/12/12/maven-checkstyle-and-eclipse/

          Show
          jeagles Jonathan Eagles added a comment - I hate to comment on this late. Can we please follow the multi-model example method documented by the checkstyle maven plugin. This will allow us to retain the one definition of checkstyle plugins as well as make it play nicely with eclipse. http://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html https://jgonian.wordpress.com/2010/12/12/maven-checkstyle-and-eclipse/
          Hide
          jeagles Jonathan Eagles added a comment -

          This effectively creates a reusable resource module (yes another module) we we can host resources that can be shared and depended on by checkstyle and eclipse.

          Show
          jeagles Jonathan Eagles added a comment - This effectively creates a reusable resource module (yes another module) we we can host resources that can be shared and depended on by checkstyle and eclipse.
          Hide
          jeagles Jonathan Eagles added a comment -

          In addition, can we consider updating the checkstyle version the maven checkstyle plugin uses. This will allow us to pick up all the bug fixes
          http://checkstyle.sourceforge.net/releasenotes.html

          <checkstyle.version>6.6</checkstyle.version>
                    <dependencies>
                      <dependency>
                        <groupId>com.puppycrawl.tools</groupId>
                        <artifactId>checkstyle</artifactId>
                        <version>${checkstyle.version}</version>
                      </dependency>
                    </dependencies>
          
          Show
          jeagles Jonathan Eagles added a comment - In addition, can we consider updating the checkstyle version the maven checkstyle plugin uses. This will allow us to pick up all the bug fixes http://checkstyle.sourceforge.net/releasenotes.html <checkstyle.version>6.6</checkstyle.version> <dependencies> <dependency> <groupId>com.puppycrawl.tools</groupId> <artifactId>checkstyle</artifactId> <version>${checkstyle.version}</version> </dependency> </dependencies>
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Thanks for all suggestions. Here is updated patch 002.

          Show
          jira.shegalov Gera Shegalov added a comment - Thanks for all suggestions. Here is updated patch 002.
          Hide
          jeagles Jonathan Eagles added a comment -

          From a high level I would still like to see the checkstyle.xml file under the hadoop-build-tools src/main/resources. Perhaps that explains the build failure below.

          Couple of minor nits below.
          can we remove azure maven-checkstyle-plugin definition?

                <plugin>
                    <groupId>org.apache.maven.plugins</groupId>
                    <artifactId>maven-checkstyle-plugin</artifactId>
                    <configuration>
                      <!-- To run with the default Sun ruleset, 
                           comment out the configLocation line -->
                      <configLocation>src/config/checkstyle.xml</configLocation>
                    </configuration>
                    
                </plugin>
          

          Looks like a few typos

          hadoop-build-tools/pom.xml
            <name>Apache Hadoop Build Toolst</name>
          

          This is the first reference to dependency of hadoop artifact by version. I think we are safe without this.

          pom.xml
                      <dependency>
                        <groupId>org.apache.hadoop</groupId>
                        <artifactId>hadoop-build-tools</artifactId>
                        <version>3.0.0-SNAPSHOT</version>
                      </dependency>
          

          the mvn checkstyle:checkstyle goal fails to the checkstyle/checkstyle.xml file being in the incorrect location

          pom.xml
                    <configuration>
                      <configLocation>checkstyle/checkstyle.xml</configLocation>
                      <failOnViolation>false</failOnViolation>
                      <format>xml</format>
                      <format>html</format>
                      <outputFile>${project.build.directory}/test/checkstyle-errors.xml</outputFile>
                     </configuration>
          
          Show
          jeagles Jonathan Eagles added a comment - From a high level I would still like to see the checkstyle.xml file under the hadoop-build-tools src/main/resources. Perhaps that explains the build failure below. Couple of minor nits below. can we remove azure maven-checkstyle-plugin definition? <plugin> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-checkstyle-plugin</artifactId> <configuration> <!-- To run with the default Sun ruleset, comment out the configLocation line --> <configLocation>src/config/checkstyle.xml</configLocation> </configuration> </plugin> Looks like a few typos hadoop-build-tools/pom.xml <name>Apache Hadoop Build Toolst</name> This is the first reference to dependency of hadoop artifact by version. I think we are safe without this. pom.xml <dependency> <groupId>org.apache.hadoop</groupId> <artifactId>hadoop-build-tools</artifactId> <version>3.0.0-SNAPSHOT</version> </dependency> the mvn checkstyle:checkstyle goal fails to the checkstyle/checkstyle.xml file being in the incorrect location pom.xml <configuration> <configLocation>checkstyle/checkstyle.xml</configLocation> <failOnViolation> false </failOnViolation> <format>xml</format> <format>html</format> <outputFile>${project.build.directory}/test/checkstyle-errors.xml</outputFile> </configuration>
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Thanks for reviewing the patch, Jonathan Eagles!

          From a high level I would still like to see the checkstyle.xml file under the hadoop-build-tools src/main/resources. Perhaps that explains the build failure below.

          That's already the case, see patch:

          rename from hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          rename to hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          

          can we remove azure maven-checkstyle-plugin definition?

          Meant to do this, but they have different rule sets.

          [tw-mbp-gshegalov hadoop-common (HADOOP-11889)]$ diff -u ./hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml ./hadoop-tools/hadoop-azure/src/config/checkstyle.xml | diffstat 
           checkstyle.xml |  138 ++++++++++++++++++++++++++++-----------------------------
           1 file changed, 69 insertions(+), 69 deletions(-)
          

          the mvn checkstyle:checkstyle goal fails to the checkstyle/checkstyle.xml file being in the incorrect location

          the location is correct. I forgot to remove RedundantThrows (not supported since 6.2) from azure's checkstyle.xml the same way I dropped it from the main one.

          Looks like a few typos

          fixed

          This is the first reference to dependency of hadoop artifact by version. I think we are safe without this.

          fixed

          Please review 003 patch.

          Show
          jira.shegalov Gera Shegalov added a comment - Thanks for reviewing the patch, Jonathan Eagles ! From a high level I would still like to see the checkstyle.xml file under the hadoop-build-tools src/main/resources. Perhaps that explains the build failure below. That's already the case, see patch: rename from hadoop-common-project/hadoop-common/dev-support/checkstyle.xml rename to hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml can we remove azure maven-checkstyle-plugin definition? Meant to do this, but they have different rule sets. [tw-mbp-gshegalov hadoop-common (HADOOP-11889)]$ diff -u ./hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml ./hadoop-tools/hadoop-azure/src/config/checkstyle.xml | diffstat checkstyle.xml | 138 ++++++++++++++++++++++++++++----------------------------- 1 file changed, 69 insertions(+), 69 deletions(-) the mvn checkstyle:checkstyle goal fails to the checkstyle/checkstyle.xml file being in the incorrect location the location is correct. I forgot to remove RedundantThrows (not supported since 6.2) from azure's checkstyle.xml the same way I dropped it from the main one. Looks like a few typos fixed This is the first reference to dependency of hadoop artifact by version. I think we are safe without this. fixed Please review 003 patch.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 39s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          -1 javac 0m 10s The patch appears to cause the build to fail.



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729670/HADOOP-11889.003.patch
          Optional Tests javadoc javac unit
          git revision trunk / 87e9978
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6230/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 39s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 javac 0m 10s The patch appears to cause the build to fail. Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729670/HADOOP-11889.003.patch Optional Tests javadoc javac unit git revision trunk / 87e9978 Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6230/console This message was automatically generated.
          Hide
          jeagles Jonathan Eagles added a comment -

          One last thing. This fails to compile the azure module do to the newly added hadoop-build-tools

          mvn clean install -DskipTests -Dmaven.java.skip
          966 [INFO] Scanning for projects...
          2527 [ERROR] The build could not read 1 project -> [Help 1]
          2527 [ERROR]
          2528 [ERROR]   The project org.apache.hadoop:hadoop-azure:3.0.0-SNAPSHOT (/Users/jeagles/hadoop/commit/hadoop/hadoop-tools/hadoop-azure/pom.xml) has 1 error
          2528 [ERROR]     'build.plugins.plugin[org.apache.maven.plugins:maven-checkstyle-plugin].dependencies.dependency.version' for org.apache.hadoop:hadoop-build-tools:jar is missing. @ org.apache.hadoop:hadoop-main:3.0.0-SNAPSHOT, /Users/jeagles/hadoop/commit/hadoop/pom.xml, line 303, column 25
          2528 [ERROR]
          2528 [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
          2528 [ERROR] Re-run Maven using the -X switch to enable full debug logging.
          2528 [ERROR]
          2528 [ERROR] For more information about the errors and possible solutions, please read the following articles:
          2528 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
          

          Also, after I remove the checkstyle plugin from the azure module I build and then run checkstyle and get this error

          mvn checkstyle:checkstyle
          3105 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.15:checkstyle (default-cli) on project hadoop-main: Execution default-cli of goal org.apache.maven.plugins:maven-checkstyle-plugin:2.15:checkstyle failed: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:2.15 or one of its dependencies could not be resolved: Failure to find org.apache.hadoop:hadoop-build-tools:jar: in http://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
          
          Show
          jeagles Jonathan Eagles added a comment - One last thing. This fails to compile the azure module do to the newly added hadoop-build-tools mvn clean install -DskipTests -Dmaven.java.skip 966 [INFO] Scanning for projects... 2527 [ERROR] The build could not read 1 project -> [Help 1] 2527 [ERROR] 2528 [ERROR] The project org.apache.hadoop:hadoop-azure:3.0.0-SNAPSHOT (/Users/jeagles/hadoop/commit/hadoop/hadoop-tools/hadoop-azure/pom.xml) has 1 error 2528 [ERROR] 'build.plugins.plugin[org.apache.maven.plugins:maven-checkstyle-plugin].dependencies.dependency.version' for org.apache.hadoop:hadoop-build-tools:jar is missing. @ org.apache.hadoop:hadoop-main:3.0.0-SNAPSHOT, /Users/jeagles/hadoop/commit/hadoop/pom.xml, line 303, column 25 2528 [ERROR] 2528 [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch . 2528 [ERROR] Re-run Maven using the -X switch to enable full debug logging. 2528 [ERROR] 2528 [ERROR] For more information about the errors and possible solutions, please read the following articles: 2528 [ERROR] [Help 1] http: //cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException Also, after I remove the checkstyle plugin from the azure module I build and then run checkstyle and get this error mvn checkstyle:checkstyle 3105 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.15:checkstyle ( default -cli) on project hadoop-main: Execution default -cli of goal org.apache.maven.plugins:maven-checkstyle-plugin:2.15:checkstyle failed: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:2.15 or one of its dependencies could not be resolved: Failure to find org.apache.hadoop:hadoop-build-tools:jar: in http: //repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 36s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          -1 javac 0m 10s The patch appears to cause the build to fail.



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729670/HADOOP-11889.003.patch
          Optional Tests javadoc javac unit
          git revision trunk / 98a6176
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6232/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 36s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 javac 0m 10s The patch appears to cause the build to fail. Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729670/HADOOP-11889.003.patch Optional Tests javadoc javac unit git revision trunk / 98a6176 Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6232/console This message was automatically generated.
          Hide
          jeagles Jonathan Eagles added a comment -

          I looked into this further. The maven-checkstyle-plugin was added to build/pluginManagement but not to build/plugins. This won't change anything for mvn clean install since checkstyle is tied to the mvn site goal. If you remove the maven-checkstyle-plugin definitions from azure and add a build plugin to the top level pom for maven-checkstyle-plugin and add back the version $

          {project.version}

          , then it will work.

          Show
          jeagles Jonathan Eagles added a comment - I looked into this further. The maven-checkstyle-plugin was added to build/pluginManagement but not to build/plugins. This won't change anything for mvn clean install since checkstyle is tied to the mvn site goal. If you remove the maven-checkstyle-plugin definitions from azure and add a build plugin to the top level pom for maven-checkstyle-plugin and add back the version $ {project.version} , then it will work.
          Hide
          jira.shegalov Gera Shegalov added a comment -

          004 patch:

          • adding project.version to build-toolls dependency
          • adding reference to checkstyle plugin in build lifecycle

          I would like not to deal with azure in this JIRA. This module uses a distinctly different checkstyle.xml. If that's no longer required we can discuss removing this override in a different JIRA.

          Show
          jira.shegalov Gera Shegalov added a comment - 004 patch: adding project.version to build-toolls dependency adding reference to checkstyle plugin in build lifecycle I would like not to deal with azure in this JIRA. This module uses a distinctly different checkstyle.xml. If that's no longer required we can discuss removing this override in a different JIRA.
          Hide
          hadoopqa Hadoop QA added a comment -

          The patch artifact directory has been removed!
          This is a fatal error for test-patch.sh. Aborting.
          Jenkins (node H3) information at https://builds.apache.org/job/PreCommit-HADOOP-Build/6233/ may provide some hints.

          Show
          hadoopqa Hadoop QA added a comment - The patch artifact directory has been removed! This is a fatal error for test-patch.sh. Aborting. Jenkins (node H3) information at https://builds.apache.org/job/PreCommit-HADOOP-Build/6233/ may provide some hints.
          Hide
          jeagles Jonathan Eagles added a comment -

          +1 pending hadoop qa. patch 004 is working sufficiently well for me. i have kicked the build since the error before seems spurious.

          Show
          jeagles Jonathan Eagles added a comment - +1 pending hadoop qa. patch 004 is working sufficiently well for me. i have kicked the build since the error before seems spurious.
          Hide
          aw Allen Wittenauer added a comment -

          Pretty much anything that touches hadoop-hdfs or is in the queue after hadoop-hdfs is subject to random failures like the above. It's only going to get worse as hadoop-hdfs gets closer and closer to the Jenkins time limit.

          Show
          aw Allen Wittenauer added a comment - Pretty much anything that touches hadoop-hdfs or is in the queue after hadoop-hdfs is subject to random failures like the above. It's only going to get worse as hadoop-hdfs gets closer and closer to the Jenkins time limit.
          Hide
          jeagles Jonathan Eagles added a comment -

          Agreed, I don't think there is much value in waiting for the build results on this patch.

          Show
          jeagles Jonathan Eagles added a comment - Agreed, I don't think there is much value in waiting for the build results on this patch.
          Hide
          jeagles Jonathan Eagles added a comment -

          Thanks, Gera Shegalov. Committed to trunk and branch-2

          Show
          jeagles Jonathan Eagles added a comment - Thanks, Gera Shegalov . Committed to trunk and branch-2
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #7713 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7713/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-project-dist/pom.xml
          • hadoop-project/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          • hadoop-build-tools/pom.xml
          • pom.xml
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7713 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7713/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) hadoop-tools/hadoop-distcp/pom.xml hadoop-project-dist/pom.xml hadoop-project/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl hadoop-tools/hadoop-azure/src/config/checkstyle.xml hadoop-build-tools/pom.xml pom.xml hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 35s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 javac 7m 25s There were no new javac warning messages.
          +1 javadoc 9m 34s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 install 1m 33s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 common tests 23m 25s Tests passed in hadoop-common.
          +1 tools/hadoop tests 1m 8s Tests passed in hadoop-azure.
          +1 tools/hadoop tests 6m 6s Tests passed in hadoop-distcp.
          +1 hdfs tests 165m 36s Tests passed in hadoop-hdfs.
              230m 21s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729733/HADOOP-11889.004.patch
          Optional Tests javadoc javac unit
          git revision trunk / 1b3b9e5
          hadoop-common test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-common.txt
          hadoop-azure test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-azure.txt
          hadoop-distcp test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-distcp.txt
          hadoop-hdfs test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-hdfs.txt
          Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/testReport/
          Java 1.7.0_55
          uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 35s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 25s There were no new javac warning messages. +1 javadoc 9m 34s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 common tests 23m 25s Tests passed in hadoop-common. +1 tools/hadoop tests 1m 8s Tests passed in hadoop-azure. +1 tools/hadoop tests 6m 6s Tests passed in hadoop-distcp. +1 hdfs tests 165m 36s Tests passed in hadoop-hdfs.     230m 21s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729733/HADOOP-11889.004.patch Optional Tests javadoc javac unit git revision trunk / 1b3b9e5 hadoop-common test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-common.txt hadoop-azure test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-azure.txt hadoop-distcp test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-distcp.txt hadoop-hdfs test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/artifact/patchprocess/testrun_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/testReport/ Java 1.7.0_55 uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6237/console This message was automatically generated.
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Thanks, Allen Wittenauer, Chris Nauroth, and Jonathan Eagles for reviews, and commit. Quite brave of you committing before Jenkins' report

          Show
          jira.shegalov Gera Shegalov added a comment - Thanks, Allen Wittenauer , Chris Nauroth , and Jonathan Eagles for reviews, and commit. Quite brave of you committing before Jenkins' report
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 35s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 javac 7m 30s There were no new javac warning messages.
          +1 javadoc 9m 37s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 install 1m 34s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 common tests 23m 50s Tests passed in hadoop-common.
          +1 tools/hadoop tests 1m 8s Tests passed in hadoop-azure.
          +1 tools/hadoop tests 6m 5s Tests passed in hadoop-distcp.
          -1 hdfs tests 163m 39s Tests failed in hadoop-hdfs.
              228m 59s  



          Reason Tests
          Failed unit tests hadoop.hdfs.TestAppendSnapshotTruncate



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729782/HADOOP-11889.004.patch
          Optional Tests javadoc javac unit
          git revision trunk / 279958b
          hadoop-common test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-common.txt
          hadoop-azure test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-azure.txt
          hadoop-distcp test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-distcp.txt
          hadoop-hdfs test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-hdfs.txt
          Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/testReport/
          Java 1.7.0_55
          uname Linux asf909.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 35s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 30s There were no new javac warning messages. +1 javadoc 9m 37s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 34s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 common tests 23m 50s Tests passed in hadoop-common. +1 tools/hadoop tests 1m 8s Tests passed in hadoop-azure. +1 tools/hadoop tests 6m 5s Tests passed in hadoop-distcp. -1 hdfs tests 163m 39s Tests failed in hadoop-hdfs.     228m 59s   Reason Tests Failed unit tests hadoop.hdfs.TestAppendSnapshotTruncate Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729782/HADOOP-11889.004.patch Optional Tests javadoc javac unit git revision trunk / 279958b hadoop-common test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-common.txt hadoop-azure test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-azure.txt hadoop-distcp test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-distcp.txt hadoop-hdfs test log https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/artifact/patchprocess/testrun_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/testReport/ Java 1.7.0_55 uname Linux asf909.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/6238/console This message was automatically generated.
          Hide
          arpitagarwal Arpit Agarwal added a comment -

          Is it just me or has this commit broken surefire?

          $ mvn test
          ...
          [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test) on project hadoop-build-tools: No tests were executed!  (Set -DfailIfNoTests=false to ignore this 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-build-tools
          
          Show
          arpitagarwal Arpit Agarwal added a comment - Is it just me or has this commit broken surefire? $ mvn test ... [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test ( default -test) on project hadoop-build-tools: No tests were executed! (Set -DfailIfNoTests= false to ignore this 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-build-tools
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Sort of, we need to add failIfNoTests=false to hadoop-build-tools pom. You can specify it on the command line in the meantime

          Show
          jira.shegalov Gera Shegalov added a comment - Sort of, we need to add failIfNoTests=false to hadoop-build-tools pom. You can specify it on the command line in the meantime
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Filed a follow-up JIRA HADOOP-11900. Thanks Arpit Agarwal!

          Show
          jira.shegalov Gera Shegalov added a comment - Filed a follow-up JIRA HADOOP-11900 . Thanks Arpit Agarwal !
          Hide
          arpitagarwal Arpit Agarwal added a comment -

          Thanks for the quick response Gera Shegalov.

          Show
          arpitagarwal Arpit Agarwal added a comment - Thanks for the quick response Gera Shegalov .
          Hide
          jira.shegalov Gera Shegalov added a comment -

          Added missing cherry-pick to branch-2.

          Show
          jira.shegalov Gera Shegalov added a comment - Added missing cherry-pick to branch-2.
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #181 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/181/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-project/pom.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          • pom.xml
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          • hadoop-project-dist/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          • hadoop-build-tools/pom.xml
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #181 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/181/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) hadoop-tools/hadoop-distcp/pom.xml hadoop-project/pom.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl pom.xml hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-tools/hadoop-azure/src/config/checkstyle.xml hadoop-project-dist/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl hadoop-build-tools/pom.xml
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #915 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/915/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • hadoop-project-dist/pom.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          • hadoop-project/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          • hadoop-build-tools/pom.xml
          • pom.xml
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #915 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/915/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) hadoop-project-dist/pom.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl hadoop-project/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl hadoop-build-tools/pom.xml pom.xml hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-tools/hadoop-distcp/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-tools/hadoop-azure/src/config/checkstyle.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #172 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/172/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • pom.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          • hadoop-project/pom.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-project-dist/pom.xml
          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-build-tools/pom.xml
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #172 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/172/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) pom.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl hadoop-project/pom.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-project-dist/pom.xml hadoop-tools/hadoop-distcp/pom.xml hadoop-build-tools/pom.xml hadoop-tools/hadoop-azure/src/config/checkstyle.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk #2113 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2113/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-project/pom.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          • hadoop-project-dist/pom.xml
          • hadoop-build-tools/pom.xml
          • pom.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #2113 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2113/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-project/pom.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl hadoop-tools/hadoop-azure/src/config/checkstyle.xml hadoop-project-dist/pom.xml hadoop-build-tools/pom.xml pom.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-tools/hadoop-distcp/pom.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #182 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/182/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • hadoop-project-dist/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          • pom.xml
          • hadoop-build-tools/pom.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-project/pom.xml
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #182 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/182/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) hadoop-project-dist/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-tools/hadoop-distcp/pom.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl hadoop-tools/hadoop-azure/src/config/checkstyle.xml pom.xml hadoop-build-tools/pom.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-project/pom.xml
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2131 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2131/)
          HADOOP-11889. Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29)

          • hadoop-tools/hadoop-distcp/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml
          • hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl
          • hadoop-project/pom.xml
          • hadoop-build-tools/pom.xml
          • hadoop-project-dist/pom.xml
          • hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl
          • hadoop-common-project/hadoop-common/dev-support/checkstyle.xml
          • hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl
          • pom.xml
          • hadoop-tools/hadoop-azure/src/config/checkstyle.xml
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2131 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2131/ ) HADOOP-11889 . Make checkstyle runnable from root project (Gera Shegalov via jeagles) (jeagles: rev 2d7363b27360e36fdd62546c0f9d0b1d78133f29) hadoop-tools/hadoop-distcp/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle.xml hadoop-build-tools/src/main/resources/checkstyle/checkstyle-noframes-sorted.xsl hadoop-project/pom.xml hadoop-build-tools/pom.xml hadoop-project-dist/pom.xml hadoop-hdfs-project/hadoop-hdfs/dev-support/checkstyle-noframes-sorted.xsl hadoop-common-project/hadoop-common/dev-support/checkstyle.xml hadoop-common-project/hadoop-common/src/test/checkstyle-noframes-sorted.xsl pom.xml hadoop-tools/hadoop-azure/src/config/checkstyle.xml

            People

            • Assignee:
              jira.shegalov Gera Shegalov
              Reporter:
              jira.shegalov Gera Shegalov
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development