Szilard Nemeth created SUBMARINE-79:
---------------------------------------

             Summary: Cleanup MockRemoteDirectoryManager
                 Key: SUBMARINE-79
                 URL: https://issues.apache.org/jira/browse/SUBMARINE-79
             Project: Hadoop Submarine
          Issue Type: Improvement
            Reporter: Szilard Nemeth
            Assignee: Szilard Nemeth


Many methods in MockRemoteDirectoryManager are declaring unnecessary throws 
clause for IOException.

Also, there are 2 dangerous NPE candidates: 
1. MockRemoteDirectoryManager#getJobStagingArea: 

{code:java}
this.jobDir = new File(jobsParentDir.getAbsolutePath(), jobName);
{code}
If jobsParentDir is null, we could have an NPE easily.

2. MockRemoteDirectoryManager#getModelDir: 

{code:java}
File modelDir = new File(modelParentDir.getAbsolutePath(), modelName);
{code}
If modelParentDir is null, we could have an NPE easily.




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to