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

    https://github.com/apache/storm/pull/2414#discussion_r150402295
  
    --- Diff: 
storm-core/test/jvm/org/apache/storm/localizer/AsyncLocalizerTest.java ---
    @@ -62,15 +62,15 @@ public void testRequestDownloadBaseTopologyBlobs() 
throws Exception {
             final String jarKey = topoId + "-stormjar.jar";
             final String codeKey = topoId + "-stormcode.ser";
             final String confKey = topoId + "-stormconf.ser";
    -        final String stormLocal = "/tmp/storm-local/";
    -        final String stormRoot = stormLocal+topoId+"/";
    +        final String stormLocalAbsolutePath = new 
File("/tmp/storm-local/").getAbsolutePath();
    --- End diff --
    
    Use `Files.createTempFile` instead of `/tmp`.


---

Reply via email to