Skip to content

Conversation

@aynroot
Copy link
Contributor

@aynroot aynroot commented Oct 30, 2020

Should solve #2153


This change is Reviewable

@jeffwidman
Copy link
Contributor

jeffwidman commented Oct 31, 2020

I'm going to go ahead and merge so master is fixed, but @aynroot do you want to add a test for the underlying issue? IIRC, you had great test coverage originally, so I'm surprised this managed to slip through...

@aynroot
Copy link
Contributor Author

aynroot commented Nov 2, 2020

yes, I'll take care of it this week, wanted to unblock the issue as fast as possible last week without having much time

the problem I think is that the metadata method is the one currently which is not covered and is almost duplicated in the tests with a helper method :D

aynroot added a commit to aynroot/kafka-python that referenced this pull request Nov 4, 2020
Use six.viewitems instead of six.iteritems to avoid encoding problems (dpkp#2154)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants