openstack/deb-python-kafka
Code Issues Proposed changes

Use six.string_types for topic type check in #824

Browse Source
This commit is contained in:
Dana Powers
2016-11-20 12:44:55 -08:00
parent 5ad6f52a80
commit c4a6e1aa68
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
kafka/consumer/subscription_state.py
View File

@@ -133,7 +133,7 @@ class SubscriptionState(object):
if self._user_assignment:
raise IllegalStateError(self._SUBSCRIPTION_EXCEPTION_MESSAGE)
if isinstance(topics, str):
if isinstance(topics, six.string_types):
topics = [topics]
if self.subscription == set(topics):
@@ -141,7 +141,7 @@ class SubscriptionState(object):
topics)
return
if any(not isinstance(t, str) for t in topics):
if any(not isinstance(t, six.string_types) for t in topics):
raise TypeError('All topics must be strings')
log.info('Updating subscribed topics to: %s', topics)