[jira] [Commented] (TEZ-2799) SimpleHistoryParser NPE

2015-09-10 Thread TezQA (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14738843#comment-14738843 ] TezQA commented on TEZ-2799: {color:red}-1 overall{color}. Here are the results of testing the latest

[jira] [Commented] (TEZ-2799) SimpleHistoryParser NPE

2015-09-09 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14737828#comment-14737828 ] Bikas Saha commented on TEZ-2799: - java.lang.NullPointerException at

[jira] [Commented] (TEZ-2799) SimpleHistoryParser NPE

2015-09-09 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14737830#comment-14737830 ] Bikas Saha commented on TEZ-2799: - Looks like a simple null check is missing, but I could be wrong. There

[jira] [Commented] (TEZ-2799) SimpleHistoryParser NPE

2015-09-09 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14737829#comment-14737829 ] Bikas Saha commented on TEZ-2799: - hadoop jar tez-0.8.1-SNAPSHOT/tez-job-analyzer-0.8.1-SNAPSHOT.jar

[jira] [Commented] (TEZ-2799) SimpleHistoryParser NPE

2015-09-09 Thread Bikas Saha (JIRA)
[ https://issues.apache.org/jira/browse/TEZ-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14737995#comment-14737995 ] Bikas Saha commented on TEZ-2799: - lgtm. minor. if {==null} else {} may be better than if {==null} if