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

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

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

    https://github.com/apache/storm/pull/1507#discussion_r69813543
  
    --- Diff: storm-core/src/jvm/org/apache/storm/Config.java ---
    @@ -1835,6 +1835,21 @@
         public static final String TOPOLOGY_CLASSPATH="topology.classpath";
     
         /**
    +     * Topology-specific classpath for the worker child process. This will 
be *prepended* to
    +     * the usual classpath, meaning it can override the Storm classpath. 
This is for debugging
    +     * purposes, and is disabled by default. To allow topologies to be 
submitted with user-first
    +     * classpaths, set the user.classpath.first.enabled config to true.
    +     */
    +    @isStringOrStringList
    +    public static final String 
TOPOLOGY_CLASSPATH_FIRST="topology.classpath.first";
    --- End diff --
    
    I agree with @satishd the name is strange. Maybe 
`topology.classpath.beginning`?


> Add ability for worker-first classpath
> --------------------------------------
>
>                 Key: STORM-1916
>                 URL: https://issues.apache.org/jira/browse/STORM-1916
>             Project: Apache Storm
>          Issue Type: Improvement
>            Reporter: Kyle Nusbaum
>            Assignee: Kyle Nusbaum
>
> Debugging in multi-tenant environments can be difficult, and having the 
> ability to override the classpath can be helpful. We want a topology config 
> that specifies an overriding classpath for a given topology. This config is 
> disabled by default since users overriding the classpath is dangerous in a 
> multi-tenant environment. The config can be temporarily enabled on Nimbus to 
> allow submission of topologies with overridden classpaths. 



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

Reply via email to