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

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

GitHub user alexpanov opened a pull request:

    https://github.com/apache/storm/pull/707

    Fix STORM-1013

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/alexpanov/storm master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/707.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #707
    
----
commit 6a54fdb2e617723ca36a82258ba562fd7fadb29f
Author: Alex Panov <[email protected]>
Date:   2015-08-28T15:35:39Z

    Fix STORM-1013

----


> [storm-elasticsearch] Expose TransportClient configuration Map to EsConfig
> --------------------------------------------------------------------------
>
>                 Key: STORM-1013
>                 URL: https://issues.apache.org/jira/browse/STORM-1013
>             Project: Apache Storm
>          Issue Type: Improvement
>            Reporter: Jungtaek Lim
>            Assignee: Alex Panov
>            Priority: Minor
>
> storm-elasticsearch has hardcoded configuration for TransportClient, 
> client.transport.sniff = true.
> Users may want to change this value, and may also want to add another 
> configuration.
> (Please refer 
> https://www.elastic.co/guide/en/elasticsearch/client/java-api/current/transport-client.html)
> If we want to pass configurations to TransportClient, I think it would be 
> better to expose Map from EsConfig, and put all things inside Map to 
> ImmutableSettings.settingsBuilder().



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

Reply via email to