search

druid-io / pydruid

A Python connector for Druid
Other
508 stars 199 forks source link

add missing sys import #20

Closed graphaelli closed 9 years ago

graphaelli commented 9 years ago

used on https://github.com/metamx/pydruid/blob/310dd7057be8d5cc1af9c54a508ebfa060161a09/pydruid/client.py#L125

griffy commented 9 years ago

I just put up a PR (https://github.com/metamx/pydruid/pull/21) that addresses this and a few other bugs I introduced when adding Python 3 support if you'd like to review it

graphaelli commented 9 years ago

Great, I'll close this if that gets merged first. I'll try to get a review in too.

xvrl commented 9 years ago

@graphaelli @drcrallen is this still needed, now that #21 is about to get merged?

xvrl commented 9 years ago

@graphaelli after merging #21, client.py already imports sys, so I think we can safely close this.

raphaellig commented 9 years ago

thanks!