mahout-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MAHOUT-807) Wrong prefixes in PrefixAdditionFilter
Date Wed, 14 Sep 2011 15:29:11 GMT

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

Hudson commented on MAHOUT-807:
-------------------------------

Integrated in Mahout-Quality #1037 (See [https://builds.apache.org/job/Mahout-Quality/1037/])
    MAHOUT-807 fix recursion

srowen : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1170494
Files : 
* /mahout/trunk/integration/src/main/java/org/apache/mahout/text/PrefixAdditionFilter.java


> Wrong prefixes in PrefixAdditionFilter
> --------------------------------------
>
>                 Key: MAHOUT-807
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-807
>             Project: Mahout
>          Issue Type: Bug
>          Components: Clustering
>    Affects Versions: 0.6
>            Reporter: Florian Bausch
>            Assignee: Sean Owen
>            Priority: Minor
>              Labels: patch
>             Fix For: 0.6
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> The org.apache.mahout.text.PrefixAdditionFilter is not working as intended. The prefix
is computed wrong when there is recursion (this is when there is at least one subdirectory).
> I think the line:
> fs.listStatus(fst.getPath(), new PrefixAdditionFilter(getConf(), getPrefix() + Path.SEPARATOR
+ current.getName(), getOptions(), writer, fs));
> should better be:
> fs.listStatus(fst.getPath(), new PrefixAdditionFilter(getConf(), getPrefix() + Path.SEPARATOR
+ current.getName() + Path.SEPARATOR + fst.getPath().getName(), getOptions(), writer, fs));

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

        

Mime
View raw message