[
https://issues.apache.org/jira/browse/MAHOUT-746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13056447#comment-13056447
]
Sean Owen commented on MAHOUT-746:
----------------------------------
+1 from me, I think this is not only a good change but well executed. It is
also something I think needed to get done for 1.0, as it gets rid of
deprecations and increases standardization, so a bonus point for that.
> Refactoring of the parallel Naive Bayes implementation in
> org.apache.mahout.classifier.naivebayes
> -------------------------------------------------------------------------------------------------
>
> Key: MAHOUT-746
> URL: https://issues.apache.org/jira/browse/MAHOUT-746
> Project: Mahout
> Issue Type: Improvement
> Components: Classification
> Affects Versions: 0.6
> Reporter: Sebastian Schelter
> Assignee: Sebastian Schelter
> Fix For: 0.6
>
> Attachments: MAHOUT-746.patch
>
>
> I refactored the code in org.apache.mahout.classifier.naivebayes to extend
> AbstractJob, decoupled the model serialization from the job output, extracted
> trainer classes and tried to clarify naming and reduce code complexity. I
> also added tests for the training M/R code as well as a toy integration test.
> It would be great if someone could review my patch to make sure I didn't
> break anything.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
