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

Sean Owen commented on MAHOUT-637:
----------------------------------

Charset.forName() won't throw a (checked) exception versus some other similar 
older methods. But yeah the Guava helper class there is shorter and avoids 
typos ("UFT-8"), and if it's going to throw an error will do so much earlier. I 
think this import/dependency earns its keep.

I can easily search-and-replace for this now or later.

> Remove direct HBase dependency
> ------------------------------
>
>                 Key: MAHOUT-637
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-637
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Classification
>    Affects Versions: 0.4
>            Reporter: Sean Owen
>            Assignee: Sean Owen
>            Priority: Minor
>              Labels: bayesian, gora, hbase
>             Fix For: 0.5
>
>         Attachments: MAHOUT-637.patch
>
>
> As discussed on the mailing list, seems desirable to remove the direct 
> dependence on HBase for now. The integration only exists for the Naive Bayes 
> Classifier, and is based on an old version. A more comprehensive strategy for 
> integrating with data sources, such as via Gora, is viewed as a desirable 
> goal for later. This is a step in that direction.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to