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

Sean Owen commented on MAHOUT-480:
----------------------------------

Good one, though the thrust of my message on the dev@ thread was that we ought 
to do argument checking across the entire codebase this way, or not. I don't 
want to use Guava in 5 classes only.

But I am at the least happy to make sure that the behavior of the code matches 
what's in this patch.

> Replace manual precondition checking with Precondition utility class from 
> Guava
> -------------------------------------------------------------------------------
>
>                 Key: MAHOUT-480
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-480
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Clustering
>            Reporter: Eugen Paraschiv
>            Priority: Minor
>             Fix For: 0.4
>
>         Attachments: MAHOUT-480.patch
>
>
> Replace manual precondition checking with the Precondition class. This will 
> affect the following classes: 
> FileDataModel
> MemoryDiffStorage
> SlopeOneRecommender
> FileDiffStorage
> AbstractJDBCDiffStorage

-- 
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