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

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

Github user harshach commented on the pull request:

    https://github.com/apache/storm/pull/1239#issuecomment-199345682
  
    @abhishekagarwal87 it shouldn't be external/sql/storm-sql-core we've extlib 
dir, why not add there.


> 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