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.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to