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 2015/04/17 20:34:58 UTC

[jira] [Commented] (STORM-791) Storm UI displays maps in the config incorrectly

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

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

GitHub user revans2 opened a pull request:

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

    STORM-791: Storm UI displays maps in the config incorrectly

    

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

    $ git pull https://github.com/revans2/incubator-storm STORM-791

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

    https://github.com/apache/storm/pull/528.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 #528
    
----
commit 6bfcb2c888481555e9ae6fd93133b82117baa7b7
Author: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Date:   2015-04-17T18:33:25Z

    STORM-791: Storm UI displays maps in the config incorrectly

----


> Storm UI displays maps in the config incorrectly
> ------------------------------------------------
>
>                 Key: STORM-791
>                 URL: https://issues.apache.org/jira/browse/STORM-791
>             Project: Apache Storm
>          Issue Type: Bug
>            Reporter: Robert Joseph Evans
>            Assignee: Robert Joseph Evans
>
> And it can be really confusing if a config is a list of strings or a single string with commas in it (as we ran into and had a painful time debugging).  Lets format the values as JSON, so there is no ambiguity, and do some simple formatting of it so you can see everything around it.



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