[jira] Commented: (MAHOUT-480) Replace manual precondition checking with Precondition utility class from Guava

["Sean Owen (JIRA)" <ji...@apache.org>]

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