New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add require 'logger' #61

Merged

bpot merged 1 commit into bpot:master from shinjiikeda:master

Oct 8, 2014

shinjiikeda commented Aug 27, 2014

Hi,
It is in the error in line 90 If I do not have to require the logger in script.
please fix it.


          add require 'logger'

b437813

This was referenced Sep 3, 2014

0.0.4 broken for Kafka 0.8.1.1 partitioned topics #63

Closed

Produce request failed on 0.8.1.1 cluster #64

Open

Owner

bpot commented Oct 8, 2014

wow, nice catch.

bpot added a commit that referenced this pull request


          Merge pull request #61 from shinjiikeda/master

1abca24

add require 'logger'

bpot merged commit 1abca24 into bpot:master

bpot mentioned this pull request

When no logger is given, the null logger fallback will cause a NameError #65

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment