[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15621733#comment-15621733 ] Akira Ajisaka commented on HADOOP-13603: LGTM +1, upgrading checkstyle version can be separated. > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf > Attachments: HADOOP-13603.001.patch > > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/package-info.java:23:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/privileged/MockPrivilegedOperationCaptor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.privileged;: > Line is longer than 80 characters (found 84). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/DockerRuntimeTestingUtils.java:17:package > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime;: > Line is longer than 80 characters (found 81). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/MockDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > {code} > Alternatively, we could look to restructure the packages here, but I question > what value this check really provides. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15615373#comment-15615373 ] Shane Kumpf commented on HADOOP-13603: -- No tests are included as this is a change to the checkstyle rules. I did create a dummy class in the runtime docker package and this rule worked to suppress the package line length error. I'm unsure of where the asflicense issues are coming from? Those files are not in my patch and appear unrelated? I believe this is ready for review. > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf > Attachments: HADOOP-13603.001.patch > > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/package-info.java:23:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/privileged/MockPrivilegedOperationCaptor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.privileged;: > Line is longer than 80 characters (found 84). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/DockerRuntimeTestingUtils.java:17:package > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime;: > Line is longer than 80 characters (found 81). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/MockDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > {code} > Alternatively, we could look to restructure the packages here, but I question > what value this check really provides. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15615321#comment-15615321 ] Hadoop QA commented on HADOOP-13603: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 15s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} 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. {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 7m 10s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 8s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 9s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 9s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 9s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 6s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 5s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 5s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 5s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 6s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} xml {color} | {color:green} 0m 2s{color} | {color:green} The patch has no ill-formed XML file. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 6s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 6s{color} | {color:green} hadoop-build-tools in the patch passed. {color} | | {color:red}-1{color} | {color:red} asflicense {color} | {color:red} 0m 18s{color} | {color:red} The patch generated 2 ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 9m 27s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:9560f25 | | JIRA Issue | HADOOP-13603 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12835794/HADOOP-13603.001.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit xml | | uname | Linux d15135053a7d 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 3de1302 | | Default Java | 1.8.0_101 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/10914/testReport/ | | asflicense | https://builds.apache.org/job/PreCommit-HADOOP-Build/10914/artifact/patchprocess/patch-asflicense-problems.txt | | modules | C: hadoop-build-tools U: hadoop-build-tools | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/10914/console | | Powered by | Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf > Attachments: HADOOP-13603.001.patch > > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} >
[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15615104#comment-15615104 ] Shane Kumpf commented on HADOOP-13603: -- This was discussed on the various -dev lists and no objections were raised regarding the removal of the package line length checkstyle rule. I will get a patch up soon. See the discussion here: http://mail-archives.apache.org/mod_mbox/hadoop-common-dev/201610.mbox/%3CCAOqB4e_9O-xN7iEN%2B0GAcxv26eXiEcEKg%2B3nn4ck36cbtdDjYg%40mail.gmail.com%3E > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/package-info.java:23:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/privileged/MockPrivilegedOperationCaptor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.privileged;: > Line is longer than 80 characters (found 84). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/DockerRuntimeTestingUtils.java:17:package > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime;: > Line is longer than 80 characters (found 81). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/MockDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > {code} > Alternatively, we could look to restructure the packages here, but I question > what value this check really provides. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15600943#comment-15600943 ] Akira Ajisaka commented on HADOOP-13603: I'm +1 for upgrading checkstyle version. > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf >Priority: Trivial > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/package-info.java:23:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/privileged/MockPrivilegedOperationCaptor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.privileged;: > Line is longer than 80 characters (found 84). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/DockerRuntimeTestingUtils.java:17:package > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime;: > Line is longer than 80 characters (found 81). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/MockDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > {code} > Alternatively, we could look to restructure the packages here, but I question > what value this check really provides. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15594574#comment-15594574 ] Akira Ajisaka commented on HADOOP-13603: To ignore line length checkstyle rule for package, adding the following setting in checkstyle.xml is fine. {code} {code} or we can upgrade checkstyle to 6.12 or upper to fix this issue. https://github.com/checkstyle/checkstyle/issues/2482 > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf >Priority: Trivial > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/package-info.java:23:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/privileged/MockPrivilegedOperationCaptor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.privileged;: > Line is longer than 80 characters (found 84). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/DockerRuntimeTestingUtils.java:17:package > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime;: > Line is longer than 80 characters (found 81). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/MockDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > {code} > Alternatively, we could look to restructure the packages here, but I question > what value this check really provides. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-13603) Remove package line length checkstyle rule
[ https://issues.apache.org/jira/browse/HADOOP-13603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15487631#comment-15487631 ] Shane Kumpf commented on HADOOP-13603: -- Per the comments in YARN-5628, I will start a DISCUSS thread on the -dev lists. > Remove package line length checkstyle rule > -- > > Key: HADOOP-13603 > URL: https://issues.apache.org/jira/browse/HADOOP-13603 > Project: Hadoop Common > Issue Type: Improvement > Components: build >Reporter: Shane Kumpf >Assignee: Shane Kumpf >Priority: Trivial > > The packages related to the DockerLinuxContainerRuntime all exceed the 80 > char line length limit enforced by checkstyle. This causes every build to > fail with a -1. I would like to exclude this rule from causing a failure. > {code} > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/DockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/package-info.java:23:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/privileged/MockPrivilegedOperationCaptor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.privileged;: > Line is longer than 80 characters (found 84). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/DockerRuntimeTestingUtils.java:17:package > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime;: > Line is longer than 80 characters (found 81). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/MockDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerCommandExecutor.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/runtime/docker/TestDockerContainerStatusHandler.java:17:package > > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker;: > Line is longer than 80 characters (found 88). > {code} > Alternatively, we could look to restructure the packages here, but I question > what value this check really provides. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org