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

stack commented on HBASE-4162:
------------------------------

I prefer Mikhail's patch where delete only happens when testing, not 
indiscriminately as per Ted patch (I was going to say that we shouldn't be 
using /tmp at all and I went to go fix this by trying to home the test on the 
HBaseTestingUtility.getTestDir which works fine writing the .tableinfo but then 
when we go to get the .tableinfo, we are making a new Configuration -- this is 
using defaults and therefore /tmp).

> Fix TestHRegionInfo.testGetSetOfHTD: delete /tmp/hbase-<username> if it 
> already exists.
> ---------------------------------------------------------------------------------------
>
>                 Key: HBASE-4162
>                 URL: https://issues.apache.org/jira/browse/HBASE-4162
>             Project: HBase
>          Issue Type: Test
>    Affects Versions: 0.92.0, 0.94.0
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>            Priority: Minor
>              Labels: test-patch
>             Fix For: 0.92.0, 0.94.0
>
>         Attachments: 
> 0001-Fixing-TestHRegionInfo.testGetSetOfHTD-by-removing-t.patch, 
> 0001-rev-2-of-fixing-TestHRegionInfo.testGetSetOfHTD-by-r.patch, 4162.txt
>
>
> Fix TestHRegionInfo.testGetSetOfHTD: delete /tmp/hbase-<username> if it 
> already exists. Otherwise, the directory is not created and the unit test 
> fails trying to compare stale data from the previous test run to what it 
> expects to be there.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to