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

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

Github user ptgoetz commented on the pull request:

    https://github.com/apache/incubator-storm/pull/39#issuecomment-47290556
  
    I'm actually -1 for this kind of change. For the reasons @d2r mentioned as 
well as the fact that I fell Storm should provide sane, and minimal,  defaults, 
and users should only have to override them when they need to.
    
    So I think this comes down to more of a documentation issue. I think it 
would be better to have instructions on how to enable jmx in the docs.


> Turn Nimbus JMX port on by default
> ----------------------------------
>
>                 Key: STORM-32
>                 URL: https://issues.apache.org/jira/browse/STORM-32
>             Project: Apache Storm (Incubating)
>          Issue Type: Improvement
>            Reporter: James Xu
>            Priority: Minor
>              Labels: Newbie
>
> https://github.com/nathanmarz/storm/issues/493
> This will be a change to nimbus.childopts. This makes it easy to capture JVM 
> diagnostic info.



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

Reply via email to