You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mahout.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2014/10/07 18:15:33 UTC

[jira] [Commented] (MAHOUT-1619) HighDFWordsPruner overwrites cache files

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

ASF GitHub Bot commented on MAHOUT-1619:
----------------------------------------

GitHub user bwebster opened a pull request:

    https://github.com/apache/mahout/pull/57

    MAHOUT-1619 - HighDFWordsPruner overwrites cache files

    Fix for MAHOUT-1619.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/bwebster/mahout master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/mahout/pull/57.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #57
    
----
commit 9b2abbe8b7108332276c3fdc32a95547ec034c33
Author: Burke Webster <bu...@collectiveip.com>
Date:   2014-10-07T16:13:28Z

    MAHOUT-1619 - HighDFWordsPruner overwrites cache files

----


> HighDFWordsPruner overwrites cache files
> ----------------------------------------
>
>                 Key: MAHOUT-1619
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1619
>             Project: Mahout
>          Issue Type: Bug
>    Affects Versions: 1.0, 0.9
>            Reporter: Burke Webster
>            Priority: Minor
>
> HighDFWordsPruner uses DistributedCache.setCacheFiles which will overwrite any files already in the cache.  Per the fix in MAHOUT-1498 we should be using addCacheFile, which will not overwrite existing cache files.



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