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

Robert Joseph Evans commented on STORM-1625:
--------------------------------------------

I wasn't paying attention to the sql work but we never should have included the 
sql jars in lib. It is an external library that is a layer on top of storm and 
should be treated as such.  I really would prefer to see the sql code compiled 
down into a topology jar that we can put in the distribution in a standard 
location so {{storm sql}} can find it, but it should not be on the classpath 
for everything.

> guava dependency is being shipped inside storm distribution
> -----------------------------------------------------------
>
>                 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