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

Sean Owen commented on MAHOUT-411:
----------------------------------

I'll commit a variation on this patch shortly. Mostly, it's that the patch 
rearranges imports and changes whitespace, and I'd say it's best to leave that 
as-is. I also simplified temp file handling in the test since we have a 
particular mechanism for that already. But the substance of the patch is just 
fine.

> Configurable reload interval in FileDataModel
> ---------------------------------------------
>
>                 Key: MAHOUT-411
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-411
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Collaborative Filtering
>            Reporter: Sebastian Schelter
>            Priority: Minor
>             Fix For: 0.4
>
>         Attachments: MAHOUT-411.patch
>
>
> Patch to enable the following usecase:
>  * all preferences are kept in a text file and loaded via a FileDataModel
>  * at some point in time the text file is completely overwritten with more 
> recent preferences
>  * an explicit refresh() on the FileDataModel is called
> With the current FileDataModel, one could not see the new preferences as it 
> would only have refreshed the whole file if that was modified more than one 
> minute ago. And I don't want to wait that long to trigger the explicit 
> refresh.
> The patch makes the interval configurable with keeping the old interval 
> length as default.
> Hope I understood the intention of the refreshing mechanism right.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to