Remove extra call to Leiningen

pystorm / streamparse

Run Python in Apache Storm topologies. Pythonic API, CLI tooling, and a topology DSL.

http://streamparse.readthedocs.io/

Apache License 2.0

1.5k stars 218 forks source link

Remove extra call to Leiningen #348

Closed pdulion closed 7 years ago

pdulion commented 7 years ago

I initially thought sparse was hanging when creating an uber-jar. After some research, I realized Leiningen ended up taking about 14 minutes to run, and sparse was calling it twice. My performance issue with Leiningen is another issue, but I thought I would at least push this change back.

coveralls commented 7 years ago

Coverage increased (+0.03%) to 44.5% when pulling 6e9387fcc608199d4e42c298f6df2f43a7bbffcd on pdulion:master into 6e8c4d891535a83d734419206bc52192e51872b9 on Parsely:master.

dan-blanchard commented 7 years ago

Wow, good catch! I'm not sure how that snuck by us for so long.

pdulion commented 7 years ago

My first open-source contribution. :-)