You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "László Bodor (Jira)" <ji...@apache.org> on 2020/12/15 07:42:00 UTC
[jira] [Commented] (TEZ-4215) Address findbugs warnings in
analyzers
[ https://issues.apache.org/jira/browse/TEZ-4215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17249539#comment-17249539 ]
László Bodor commented on TEZ-4215:
-----------------------------------
[~jeagles]: please take a look at my patch (see comment above)? as we're moving to github, let's have findbugs clean for that
> Address findbugs warnings in analyzers
> --------------------------------------
>
> Key: TEZ-4215
> URL: https://issues.apache.org/jira/browse/TEZ-4215
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Jonathan Turner Eagles
> Assignee: Jonathan Turner Eagles
> Priority: Major
> Attachments: TEZ-4215.01.patch, TEZ-4215.WIP.patch
>
>
> {code}
> NP Null passed for non-null parameter of com.google.common.base.Strings.isNullOrEmpty(String) in org.apache.tez.analyzer.plugins.CriticalPathAnalyzer.createCriticalPath(DagInfo, TaskAttemptInfo, long, Map)
> Bug type NP_NULL_PARAM_DEREF (click for details)
> In class org.apache.tez.analyzer.plugins.CriticalPathAnalyzer
> In method org.apache.tez.analyzer.plugins.CriticalPathAnalyzer.createCriticalPath(DagInfo, TaskAttemptInfo, long, Map)
> Called method com.google.common.base.Strings.isNullOrEmpty(String)
> Argument 1 might be null but must not be null
> Value loaded from nextAttemptId
> Method invoked at CriticalPathAnalyzer.java:[line 588]
> Known null at CriticalPathAnalyzer.java:[line 576]
> NP Null passed for non-null parameter of com.google.common.base.Strings.isNullOrEmpty(String) in org.apache.tez.analyzer.plugins.CriticalPathAnalyzer.createCriticalPath(DagInfo, TaskAttemptInfo, long, Map)
> Bug type NP_NULL_PARAM_DEREF (click for details)
> In class org.apache.tez.analyzer.plugins.CriticalPathAnalyzer
> In method org.apache.tez.analyzer.plugins.CriticalPathAnalyzer.createCriticalPath(DagInfo, TaskAttemptInfo, long, Map)
> Called method com.google.common.base.Strings.isNullOrEmpty(String)
> Argument 1 might be null but must not be null
> Value loaded from nextAttemptId
> Method invoked at CriticalPathAnalyzer.java:[line 588]
> Known null at CriticalPathAnalyzer.java:[line 519]
> NP Null passed for non-null parameter of com.google.common.base.Strings.isNullOrEmpty(String) in org.apache.tez.analyzer.plugins.CriticalPathAnalyzer.createCriticalPath(DagInfo, TaskAttemptInfo, long, Map)
> Bug type NP_NULL_PARAM_DEREF (click for details)
> In class org.apache.tez.analyzer.plugins.CriticalPathAnalyzer
> In method org.apache.tez.analyzer.plugins.CriticalPathAnalyzer.createCriticalPath(DagInfo, TaskAttemptInfo, long, Map)
> Called method com.google.common.base.Strings.isNullOrEmpty(String)
> Argument 1 might be null but must not be null
> Value loaded from nextAttemptId
> Method invoked at CriticalPathAnalyzer.java:[line 588]
> Known null at CriticalPathAnalyzer.java:[line 532]
> NP Null passed for non-null parameter of com.google.common.base.Functions.forMap(Map, Object) in org.apache.tez.analyzer.plugins.VertexLevelCriticalPathAnalyzer.sortByValues(Map)
> Bug type NP_NONNULL_PARAM_VIOLATION (click for details)
> In class org.apache.tez.analyzer.plugins.VertexLevelCriticalPathAnalyzer
> In method org.apache.tez.analyzer.plugins.VertexLevelCriticalPathAnalyzer.sortByValues(Map)
> Called method com.google.common.base.Functions.forMap(Map, Object)
> Argument 2 is definitely null but must not be null
> At VertexLevelCriticalPathAnalyzer.java:[line 116]
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)