mahout-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Robin Anil <robin.a...@gmail.com>
Subject Re: [jira] Resolved: (MAHOUT-515) PFPGrowthTest has a hard-wired reference to an existing output directory
Date Mon, 04 Oct 2010 22:17:02 GMT
The temp folder gets deleted after each test. Its not getting preserved
across tests. Thats why things are failing. Can you tell me how I can keep
the dir for the entire duration of the Test File Suite?


On Sun, Oct 3, 2010 at 7:38 PM, Jeff Eastman <jdog@windwardsolutions.com>wrote:

>  I've tried to incorporate getTestTempDirPath for the output directory but
> it causes 2 of the 3 tests to fail. I've tried debugging that for a couple
> of hours to no avail. As-is the test passes but is a waiting trap for the
> next test which may get written using the local output directory. Not a
> showstopper for 0.4 RC but I think it ought to be corrected.
>
> It should be a simple fix for someone who understands the code or who can
> look at it from a fresh perspective. The test is setting
>     params.set("output", "output/frequentpatterns");
> but then, in PFPGrowth it's doing things like
>    Path parallelCountingPath = new Path(params.get("output"),
> "parallelcounting");
> and, even
>    Path frequentPatternsPath = new Path(params.get("output"),
> "frequentPatterns");
>
> It's not fixed yet.
>
>
>
>
> On 10/3/10 6:37 AM, Sean Owen (JIRA) wrote:
>
>>      [
>> https://issues.apache.org/jira/browse/MAHOUT-515?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel]
>>
>> Sean Owen resolved MAHOUT-515.
>> ------------------------------
>>
>>       Assignee: Jeff Eastman
>>     Resolution: Fixed
>>
>> Do I understand this is resolved then? Tests run locally and on Maven for
>> me, and on Hudson AFAICT. If it's just a hygiene issues definitely go forth
>> and tweak it.
>>
>>  PFPGrowthTest has a hard-wired reference to an existing output directory
>>> ------------------------------------------------------------------------
>>>
>>>                 Key: MAHOUT-515
>>>                 URL: https://issues.apache.org/jira/browse/MAHOUT-515
>>>             Project: Mahout
>>>          Issue Type: Bug
>>>          Components: Frequent Itemset/Association Rule Mining
>>>    Affects Versions: 0.3
>>>            Reporter: Jeff Eastman
>>>            Assignee: Jeff Eastman
>>>             Fix For: 0.4
>>>
>>>
>>> This test began failing when another test was added which had the same
>>> dependency. That test has been corrected but is not yet checked-in.
>>> The correct pattern is to use getTestTempDirPath() to allocate temporary
>>> files. I was unable to quickly do this in the test (it caused other of its
>>> tests to fail) so I'm opening an issue for somebody more knowledgeable in
>>> this code.
>>>
>>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message