[ 
https://issues.apache.org/jira/browse/STORM-1399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15062726#comment-15062726
 ] 

ASF GitHub Bot commented on STORM-1399:
---------------------------------------

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

    https://github.com/apache/storm/pull/955#discussion_r47956102
  
    --- Diff: 
external/storm-hdfs/src/test/java/org/apache/storm/hdfs/blobstore/BlobStoreTest.java
 ---
    @@ -76,6 +76,7 @@
     
       @Before
       public void init() {
    +    System.setProperty("test.build.data", "target/test/data");
         initializeConfigs();
         baseFile = new File("/tmp/blob-store-test-"+UUID.randomUUID());
    --- End diff --
    
    Why are we not also changing here?


> Blobstore tests should write data to `target` so it gets removed when running 
> `mvn clean`
> -----------------------------------------------------------------------------------------
>
>                 Key: STORM-1399
>                 URL: https://issues.apache.org/jira/browse/STORM-1399
>             Project: Apache Storm
>          Issue Type: Bug
>    Affects Versions: 0.11.0
>            Reporter: P. Taylor Goetz
>            Assignee: P. Taylor Goetz
>            Priority: Minor
>
> Currently the blobstore tests create files/directories in the current working 
> directory. It would be better to use `target` so they get deleted when 
> running `mvn clean` and are ignored by git.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to