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

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

Github user haohui commented on the pull request:

    https://github.com/apache/storm/pull/1239#issuecomment-199093496
  
    Might need to run some manual tests to validate.
    
    I think there are several jars (e.g., guava) need to be bundled as well. Is 
it a good place to put them in as well?


> Move storm-sql dependencies out of lib folder
> ---------------------------------------------
>
>                 Key: STORM-1625
>                 URL: https://issues.apache.org/jira/browse/STORM-1625
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-core, storm-sql
>            Reporter: Abhishek Agarwal
>            Assignee: Abhishek Agarwal
>
> We shade guava classes inside storm-core so that client can work with any 
> version of guava without any clashes. However, storm-sql-core has a 
> transitive dependency on guava and thus the guava jar is still getting 
> shipped in lib folder. 



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

Reply via email to