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

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

Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1012#discussion_r50280610
  
    --- Diff: storm-core/src/clj/org/apache/storm/ui/core.clj ---
    @@ -863,6 +855,7 @@
                                           (.get_eventlog_host comp-page-info)
                                           (.get_eventlog_port comp-page-info)
                                           secure?)
    +       "profilingAndDebuggingCapable" (not on-windows?)
    --- End diff --
    
    Can we file a follow on JIRA to make this a bit cleaner?  Perhaps a script 
for different environments that we can check at run time if the script is there?


> Worker "profiler" actions broken by default
> -------------------------------------------
>
>                 Key: STORM-1452
>                 URL: https://issues.apache.org/jira/browse/STORM-1452
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-core
>    Affects Versions: 1.0.0
>            Reporter: Derek Dagit
>            Assignee: Derek Dagit
>            Priority: Blocker
>
> * The profiler script flight.bash is not packaged by default.
> * The default options enable the Oracle specific flight-recorder that 
> requires a support subscription.
> The option to enable the profiler should not be enabled by default.  Other 
> actions such as worker restart, debugging, and heap can remain enabled.



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

Reply via email to