Stig Rohde Døssing created STORM-3472:
-----------------------------------------

             Summary: STORM-3411 should have tests, and we shouldn't catch NPE 
for control flow
                 Key: STORM-3472
                 URL: https://issues.apache.org/jira/browse/STORM-3472
             Project: Apache Storm
          Issue Type: Improvement
          Components: storm-webapp
    Affects Versions: 2.0.1
            Reporter: Stig Rohde Døssing
            Assignee: Stig Rohde Døssing


I think the code merged in STORM-3411 should have added tests that the new 
functionality works.

We should get rid of the new bit of code that try-catches an NPE to check 
whether the downloaded file is inside a worker dir. Instead, we should move the 
name generation up the call hierarchy to a place where we can tell whether 
we're inside a worker dir or not.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to