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

Eugen Paraschiv edited comment on MAHOUT-480 at 8/14/10 6:46 AM:
-----------------------------------------------------------------

This patch covers a much wider portion of the system, in accordance with the 
feedback. The goal is to replace manual condition checking throwing 
IllegalArgumentException with Preconditions.checkArguments calls. If the 
feedback on this is OK, I will of course continue to cover the rest of the 
system and provide a second patch. 

      was (Author: eugenparaschiv):
    This patch covers a much wider portion of the system, in accordance with 
the feedback. The goal is to replace manual condition checking throwing 
IllegalArgumentException with Preconditions.checkArguments calls
  
> 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_v2.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