Github user srdo commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2414#discussion_r150402445
  
    --- Diff: 
storm-core/test/jvm/org/apache/storm/daemon/supervisor/BasicContainerTest.java 
---
    @@ -370,7 +370,7 @@ public void testLaunch() throws Exception {
             final String workerConf = ContainerTest.asAbsPath(log4jdir, 
"worker.xml");
    --- End diff --
    
    If you start the paths from `Files.createTempFile` instead of ``tmp` for 
`stormHome` and `stormLocal`, you should be able to get rid of the 
`prependFilePrefixIfOnWindows` method.


---

Reply via email to