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

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

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

    https://github.com/apache/incubator-storm/pull/28#discussion_r12282760
  
    --- Diff: storm-core/src/clj/backtype/storm/daemon/supervisor.clj ---
    @@ -438,6 +438,13 @@
           (FileUtils/moveDirectory (File. tmproot) (File. stormroot))
           ))
     
    +(defn jlp [stormroot conf]
    +  (let [resource-root (str stormroot "/" RESOURCES-SUBDIR)
    +        os (clojure.string/replace (System/getProperty "os.name") #"\s+" 
"_")
    +        arch (System/getProperty "os.arch")
    +        arch-resource-root (str resource-root "/" os "-" arch)]
    +    (str arch-resource-root ":" resource-root ":" (conf 
JAVA-LIBRARY-PATH)))) 
    --- End diff --
    
    Yes I think it would.


> ship JNI dependencies with the storm jar
> ----------------------------------------
>
>                 Key: STORM-232
>                 URL: https://issues.apache.org/jira/browse/STORM-232
>             Project: Apache Storm (Incubating)
>          Issue Type: Sub-task
>            Reporter: Robert Joseph Evans
>
> In a multi-tenant environment we need a way for users to include their JNI 
> dependencies in the storm topology jar itself.  This mostly involves a 
> convention on how to package the dependencies, and proper environment/command 
> line setup when launching the worker process.



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

Reply via email to