[jira] [Commented] (YARN-6513) Fix for FindBugs getPendingLogFilesToUpload() possible NPE
[ https://issues.apache.org/jira/browse/YARN-6513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15980168#comment-15980168 ] Hadoop QA commented on YARN-6513: - | (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} 13m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 20s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 29s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 15s{color} | {color:green} trunk passed {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 0m 56s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common in trunk has 2 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 29s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 26s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 25s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 25s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 17s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 27s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 12s{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} findbugs {color} | {color:green} 1m 0s{color} | {color:green} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2) {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 26s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 2m 16s{color} | {color:green} hadoop-yarn-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 20s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 24m 2s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:0ac17dc | | JIRA Issue | YARN-6513 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12864653/YARN-6513.01.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 1b24337661ea 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / fda86ef | | Default Java | 1.8.0_121 | | findbugs | v3.1.0-RC1 | | findbugs | https://builds.apache.org/job/PreCommit-YARN-Build/15716/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common-warnings.html | | Test Results | https://builds.apache.org/job/PreCommit-YARN-Build/15716/testReport/ | | modules | C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common | | Console output | https://builds.apache.org/job/PreCommit-YARN-Build/15716/console | | Powered by | Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > Fix for FindBugs getPendingLogFilesToUpload() possible NPE > --
[jira] [Commented] (YARN-6513) Fix for FindBugs getPendingLogFilesToUpload() possible NPE
[ https://issues.apache.org/jira/browse/YARN-6513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15981855#comment-15981855 ] Haibo Chen commented on YARN-6513: -- Thanks [~wilfreds] for the patch. One nit on the patch, we pass 0 to the HashSet constructor given that we don't reuse it. > Fix for FindBugs getPendingLogFilesToUpload() possible NPE > -- > > Key: YARN-6513 > URL: https://issues.apache.org/jira/browse/YARN-6513 > Project: Hadoop YARN > Issue Type: Bug >Reporter: Wilfred Spiegelenburg >Assignee: Wilfred Spiegelenburg > Attachments: YARN-6513.01.patch > > > {code} > Possible null pointer dereference in > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > due to return value of called method > Bug type NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE (click for details) > In class org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue > In method > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > Local variable stored in JVM register ? > Method invoked at AggregatedLogFormat.java:[line 314] > Known null at AggregatedLogFormat.java:[line 314] > {code} -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6513) Fix for FindBugs getPendingLogFilesToUpload() possible NPE
[ https://issues.apache.org/jira/browse/YARN-6513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15982205#comment-15982205 ] Naganarasimha G R commented on YARN-6513: - [~haibochen] & [~wilfreds], [~cheersyang] was already working on findbugs for yarn under umbrella jira HADOOP-14336 , just yesterday we concluded that he split yarn jira into project specific jira's and hence he created YARN-6517 - YARN-6520 and split his existing patch and uploaded. So as WeiWei yang had started first i thought of making this jira duplicate of YARN-6517. thoughts? > Fix for FindBugs getPendingLogFilesToUpload() possible NPE > -- > > Key: YARN-6513 > URL: https://issues.apache.org/jira/browse/YARN-6513 > Project: Hadoop YARN > Issue Type: Bug >Reporter: Wilfred Spiegelenburg >Assignee: Wilfred Spiegelenburg > Attachments: YARN-6513.01.patch > > > {code} > Possible null pointer dereference in > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > due to return value of called method > Bug type NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE (click for details) > In class org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue > In method > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > Local variable stored in JVM register ? > Method invoked at AggregatedLogFormat.java:[line 314] > Known null at AggregatedLogFormat.java:[line 314] > {code} -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6513) Fix for FindBugs getPendingLogFilesToUpload() possible NPE
[ https://issues.apache.org/jira/browse/YARN-6513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15983666#comment-15983666 ] Haibo Chen commented on YARN-6513: -- Yeah, I am OK with close this as a duplicate. > Fix for FindBugs getPendingLogFilesToUpload() possible NPE > -- > > Key: YARN-6513 > URL: https://issues.apache.org/jira/browse/YARN-6513 > Project: Hadoop YARN > Issue Type: Bug >Reporter: Wilfred Spiegelenburg >Assignee: Wilfred Spiegelenburg > Attachments: YARN-6513.01.patch > > > {code} > Possible null pointer dereference in > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > due to return value of called method > Bug type NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE (click for details) > In class org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue > In method > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > Local variable stored in JVM register ? > Method invoked at AggregatedLogFormat.java:[line 314] > Known null at AggregatedLogFormat.java:[line 314] > {code} -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6513) Fix for FindBugs getPendingLogFilesToUpload() possible NPE
[ https://issues.apache.org/jira/browse/YARN-6513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15984011#comment-15984011 ] Wilfred Spiegelenburg commented on YARN-6513: - Yep, I am happy with making it a duplicate, I'll comment on the change there. > Fix for FindBugs getPendingLogFilesToUpload() possible NPE > -- > > Key: YARN-6513 > URL: https://issues.apache.org/jira/browse/YARN-6513 > Project: Hadoop YARN > Issue Type: Bug >Reporter: Wilfred Spiegelenburg >Assignee: Wilfred Spiegelenburg > Attachments: YARN-6513.01.patch > > > {code} > Possible null pointer dereference in > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > due to return value of called method > Bug type NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE (click for details) > In class org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue > In method > org.apache.hadoop.yarn.logaggregation.AggregatedLogFormat$LogValue.getPendingLogFilesToUpload(File) > Local variable stored in JVM register ? > Method invoked at AggregatedLogFormat.java:[line 314] > Known null at AggregatedLogFormat.java:[line 314] > {code} -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org