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

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

GitHub user clockfly reopened a pull request:

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

    STORM-253: Allow storm worker to use dynamic port instead of pre-configed. 
Add a ne...

    PR for [STORM-253](https://issues.apache.org/jira/browse/STORM-253)
    
    Allow supervisor to decide worker dynamiclly instead of pre-configured in 
configure file. This is especially useful in a shared cluster environment.
    
    Changes:
    1. Add a new config "supervisor.slots" to set the number of workers per 
supervisor.
    2. Add class ServerSocketFactory to reserve the port for worker.
    
    
    


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

    $ git pull https://github.com/clockfly/incubator-storm worker-dynamic-port

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

    https://github.com/apache/storm/pull/45.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 #45
    
----
commit 5350a04e47deaa71890708c8719998a930e7446d
Author: Sean Zhong <[email protected]>
Date:   2014-03-06T03:36:21Z

    Allow storm worker to use dynamic port instead of pre-configed. Add a new 
config like "supervisor.slots: 3" to config the number of workers.

----


> Allow worker to choose available port instead of pre-configed
> -------------------------------------------------------------
>
>                 Key: STORM-253
>                 URL: https://issues.apache.org/jira/browse/STORM-253
>             Project: Apache Storm
>          Issue Type: Improvement
>            Reporter: Sean Zhong
>            Priority: Minor
>
> To avoid port conflict, we should allow supervisor to choose available port 
> for worker, instead of pre-configuring it in storm.yaml.
> Currently, the config for port is:
> supervisor.slots.ports: [6700,6701,6702]
> We need to obsolete it, and add a new config "supervisor.slots“. For example:
> supervisor.slots: 3
> There was a pull request from Andy Feng. We can refer to it and pull in the 
> changes of worker port first. 
> (https://github.com/nathanmarz/storm/pull/622/commits)



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

Reply via email to