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

Sean Owen commented on MAHOUT-807:
----------------------------------

Sounds good, have you verified this by testing it out? that it recurs properly 
with this change?

> 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
>            Priority: Minor
>              Labels: patch
>   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

        

Reply via email to