search

twitter / summingbird

Streaming MapReduce with Scalding and Storm
https://twitter.com/summingbird
Apache License 2.0
2.14k stars 267 forks source link

remove Warnings and unused imports #586

Closed AnishShah closed 9 years ago

AnishShah commented 9 years ago

Hi, I'm trying to remove Warnings that occur during Travis CI . And, also I removed unused imports in those files. Please tell me if any changes are required.

Thanks.

AnishShah commented 9 years ago

Travis build is not starting. Any idea ?

ianoc commented 9 years ago

Travis is pretty terribly slow for us often, it might take up to an hour for it to kick in. Thanks for the PR btw, will take a look through today

johnynek commented 9 years ago

looks like the storm tests had their usual noisy failure. Restarted.

Looks good to me.

johnynek commented 9 years ago

Can you merge with develop? I think we have the tests fixed so they are not spuriously failing.

AnishShah commented 9 years ago

Cool On 21 Feb 2015 03:18, "P. Oscar Boykin" notifications@github.com wrote:

Can you merge with develop? I think we have the tests fixed so they are not spuriously failing.

— Reply to this email directly or view it on GitHub https://github.com/twitter/summingbird/pull/586#issuecomment-75325252.