Excellent, thanks for the info! SimonC
-----Original Message----- From: P. Taylor Goetz [mailto:[email protected]] Sent: 27 August 2014 18:40 To: [email protected] Subject: Re: Consequences of STORM-447 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 >>
