You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@samoa.apache.org by gdfm <gi...@git.apache.org> on 2015/05/12 11:12:18 UTC

[GitHub] incubator-samoa pull request: SAMOA-27: Storm upgrade to the lates...

Github user gdfm commented on the pull request:

    https://github.com/apache/incubator-samoa/pull/25#issuecomment-101205275
  
    All the tests pass.
    Tested various runs of Bagging, VHT, and AMRules on Storm local model and they all work.
    Also ran the ClusteringEvaluation. Seems to work but it's hard to evaluate :)
    
    In the samoa script, given that now storm-core.jar is in the /lib directory of storm, we can simplify the line creating the class path by removing the STORM_JAR entry.
    
    +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---