[
https://issues.apache.org/jira/browse/STORM-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14196322#comment-14196322
]
ASF GitHub Bot commented on STORM-546:
--------------------------------------
GitHub user strongh opened a pull request:
https://github.com/apache/storm/pull/306
use configured local hostname for reporting metrics and errors
This is a proposed fix for
https://issues.apache.org/jira/browse/STORM-546
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/yieldbot/storm local-hostname-conf
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/306.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 #306
----
commit a370c16780d143eb756c428c4c6a26dac05de5ab
Author: Homer Strong <[email protected]>
Date: 2014-11-04T16:30:20Z
use configured local hostname for reporting metrics and errors
----
> Local hostname configuration ignored by executor
> ------------------------------------------------
>
> Key: STORM-546
> URL: https://issues.apache.org/jira/browse/STORM-546
> Project: Apache Storm
> Issue Type: Bug
> Affects Versions: 0.9.3-rc2
> Reporter: Homer Strong
> Priority: Minor
>
> The executor reports hostname using `util/memoized-local-hostname`, but
> doesn't check the configuration to see whether `STORM_LOCAL_HOSTNAME` has
> been set. Under this change the executor checks the configuration before
> falling back to guessing the local hostname.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
