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

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

Github user revans2 commented on the pull request:

    https://github.com/apache/incubator-storm/pull/60#issuecomment-39866773
  
    If I run remoteconfvalue through python I get
    ```
    $ storm remoteconfvalue ui.port
    ui.port: 8080
    ```
    
    If I call the java code directly like this patch is doing I get
    ```
     $ storm jar bogus backtype.storm.command.config_value ui.port
    ...
    VALUE: 8080
    ```
    
    The python code replaces the VALUE: with ui.port:.  Like I said I am not 
sure it is critical, but it is a difference.
    
    As for STORM-273 vs STORM-274, STORM-273 has two +1 on it so I'll just pull 
it in, and then this pull request should hopefully recognize it and you won't 
need to do anything.


> Add support for command remoteconfvalue in storm.cmd
> ----------------------------------------------------
>
>                 Key: STORM-274
>                 URL: https://issues.apache.org/jira/browse/STORM-274
>             Project: Apache Storm (Incubating)
>          Issue Type: Bug
>    Affects Versions: 0.9.2-incubating
>            Reporter: Suresh Srinivas
>         Attachments: STORM-274.patch
>
>
> Currently storm.cmd does not support remoteconfvalue as supported by its 
> linux counterpart storm.py.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to