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

ASF GitHub Bot commented on STORM-845:
--------------------------------------

Github user HeartSaVioR commented on the pull request:

    https://github.com/apache/storm/pull/573#issuecomment-125113030
  
    @sweetest 
    Please check comments which doesn't show as outdated. You seems missed some 
comments.
    Other things are fine. 
    
    And I agree that ES is not a queue so it may not feel comfortable to 
implement Spout for ES.
    
    Overall I'm fine for merging this in. 
    As @harshach want to volunteer this module, I'd like @harshach to take a 
look, too.
    When @harshach confirms it's good to merge, I'll merge to master.
    
    And please add me and @harshach as Committer Sponsors.
    You can refer 
https://github.com/apache/storm/blob/master/external/storm-redis/README.md to 
see how to add.
    
    Thanks for contributing!


> Storm ElasticSearch connector
> -----------------------------
>
>                 Key: STORM-845
>                 URL: https://issues.apache.org/jira/browse/STORM-845
>             Project: Apache Storm
>          Issue Type: New Feature
>            Reporter: Adrian Seungjin Lee
>            Assignee: Adrian Seungjin Lee
>
> It would be nice to provide storm driver for elasticsearch, just like it does 
> for hive, redis and so on.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to