Skip to content

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

Closed
clockfly wants to merge 1 commit intoapache:masterfrom
clockfly:worker-dynamic-port
Closed

STORM-253: Allow storm worker to use dynamic port instead of pre-configed. Add a ne...#45
clockfly wants to merge 1 commit intoapache:masterfrom
clockfly:worker-dynamic-port

Conversation

@clockfly
Copy link
Contributor

@clockfly clockfly commented Mar 6, 2014

PR for 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.

… new config like "supervisor.slots: 3" to config the number of workers.
@revans2
Copy link
Contributor

revans2 commented May 14, 2014

Please add the JIRA number to the title of the pull request, not just in the description.

@clockfly
Copy link
Contributor Author

Will open another PR.

@clockfly clockfly closed this May 21, 2014
@clockfly clockfly changed the title Allow storm worker to use dynamic port instead of pre-configed. Add a ne... STORM-253: Allow storm worker to use dynamic port instead of pre-configed. Add a ne... Nov 4, 2014
@clockfly clockfly reopened this Nov 4, 2014
@d2r
Copy link

d2r commented Feb 9, 2015

@clockfly, Is this pull request still valid? If so, it seems we need an up-merge. If not, can we close it?

@clockfly
Copy link
Contributor Author

it is a long time that this PR not get updated.

Let's close it for now, I prefer to create new PR if this patch is still needed.

@clockfly clockfly closed this Feb 10, 2015
Parth-Brahmbhatt pushed a commit to Parth-Brahmbhatt/incubator-storm that referenced this pull request Mar 7, 2016
BUG-44565: Increase number of workers for test topology.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants