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

ASF GitHub Bot commented on MAHOUT-1493:
----------------------------------------

Github user andrewpalumbo commented on the pull request:

    https://github.com/apache/mahout/pull/32#issuecomment-49396941
  
    One of their first questions was where to move it (out of the 
o.a.m.sparkbindings.drm package).  My suggestion, for now, was a new package in 
the spark module: o.a.m.classification.(naivebayes).   mrlegacy keeps 
classification algo packages in o.a.m.classifier.  If we wanted to keep 
consistent with naming (stick with classifiers v. classification) would this 
cause conflicts?  
    
    The next step would be to create some scalatests for this.  This should be 
relatively easy since most of the rest of mrlegacy NB has is either MR 
independent or has sequential options. So it should be somewhat straightforward 
to reproduce (some of) the mrlegacy tests. 


> Port Naive Bayes to the Spark DSL
> ---------------------------------
>
>                 Key: MAHOUT-1493
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1493
>             Project: Mahout
>          Issue Type: Bug
>          Components: Classification
>            Reporter: Sebastian Schelter
>            Assignee: Sebastian Schelter
>             Fix For: 1.0
>
>         Attachments: MAHOUT-1493.patch, MAHOUT-1493.patch, MAHOUT-1493.patch, 
> MAHOUT-1493.patch, MAHOUT-1493a.patch
>
>
> Port our Naive Bayes implementation to the new spark dsl. Shouldn't require 
> more than a few lines of code.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to