You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2014/11/04 07:59:34 UTC
[jira] [Commented] (STORM-253) Allow worker to choose available
port instead of pre-configed
[ 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 <cl...@gmail.com>
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)