remove clj-time dependency

[Brandon Adams <em...@gmail.com>]

Hi,

I noticed that clj-time is only declared in

storm-core/src/clj/backtype/storm/ui/helpers.clj

and despite being declare, it is unused. As such, I'd like to REMOVE
clj-time as a dependency from storm (and thus free up more modern versions
of clj-time and joda-time in the process).

Given that some people likely rely on clj-time/joda-time already existing
on the classpath, this probably wouldn't be eligible for a bugfix release
like 0.9.5 (if such a thing comes into being), but perhaps could make it
into 0.10.x and 0.11.x?

Would this need a jira ticket or would a simple pull request removing the
the dependency suffice?

Thanks,

Brandon Adams

Re: remove clj-time dependency

[Bobby Evans <ev...@yahoo-inc.com.INVALID>]

Yes I would love to see it go into either 0.10 or 0.11.  Please put up a pull request.
 - Bobby
 


     On Wednesday, May 27, 2015 4:35 PM, Brandon Adams <em...@gmail.com> wrote:
   

 Hi,

I noticed that clj-time is only declared in

storm-core/src/clj/backtype/storm/ui/helpers.clj

and despite being declare, it is unused. As such, I'd like to REMOVE
clj-time as a dependency from storm (and thus free up more modern versions
of clj-time and joda-time in the process).

Given that some people likely rely on clj-time/joda-time already existing
on the classpath, this probably wouldn't be eligible for a bugfix release
like 0.9.5 (if such a thing comes into being), but perhaps could make it
into 0.10.x and 0.11.x?

Would this need a jira ticket or would a simple pull request removing the
the dependency suffice?

Thanks,

Brandon Adams