GitHub user MichealShin opened a pull request:
https://github.com/apache/storm/pull/2539
STORM-2917: Check the derecated config nimbus.host
There is a situation: the deployer wants to use the new nimbus
config(nimbus.seeds), but still leave the blank deprecated config(nimbus.host)
in storm.yaml. It will not work.
To avoid this, the program should at least check whether the deprecated
config is blank before using it to connect.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/MichealShin/storm master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2539.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 #2539
----
commit a4fde2c43cb529197572c9524d0e9800c61bac34
Author: MichealShin <luochen_fei@...>
Date: 2018-01-30T09:42:38Z
STORM-2917: Check the derecated config nimbus.host
There is a situation: the deployer wants to use the new nimbus
config(nimbus.seeds), but still leave the blank deprecated config(nimbus.host)
in storm.yaml. It will not work.
To avoid this, the program should at least check whether the deprecated
config is blank.
commit 59952ef37293196b160d5854ce834c5c5bfc202a
Author: MichealShin <luochen_fei@...>
Date: 2018-01-30T09:44:29Z
Merge pull request #2 from MichealShin/MichealShin-patch-2
STORM-2917: Check the derecated config nimbus.host
----
---
