Naresh is correct.

To use the bundled Storm versions of these dependencies, you simply need to 
change import statements.

-Taylor

> On Aug 27, 2014, at 11:33 AM, Naresh Kosgi <[email protected]> wrote:
> 
> I have not tried this yet but I think you might just need to change the way
> you import them.  Instead of getting them from their regular package name
> you will need to import using the shade package name.  I believe that
> should get it working if you are using the version provided by storm.
> 
> 
>> On Wed, Aug 27, 2014 at 11:23 AM, Jon Logan <[email protected]> wrote:
>> 
>> You should be able to put the jars in the lib/ directory of the Storm
>> install, on all machines, and then mark the dependency as provided on your
>> side.
>> 
>> 
>> On Wed, Aug 27, 2014 at 11:15 AM, Simon Cooper <
>> [email protected]> wrote:
>> 
>>> I've just noticed STORM-447. We use several of storm's provided
>>> dependencies in our topology code (namely curator, guava, and zookeeper
>> via
>>> kafka). With this pull request, would we have to supply our own copy of
>>> these libraries in the topology jar? Would it be possible to still use
>>> storm's versions in our topology code to reduce the size of the uploaded
>>> jar?
>>> 
>>> Thanks,
>>> SimonC
>> 

Reply via email to