diff options
| author | Dana Powers <dana.powers@rd.io> | 2015-12-10 17:48:12 -0800 |
|---|---|---|
| committer | Dana Powers <dana.powers@rd.io> | 2015-12-10 18:37:03 -0800 |
| commit | 1cb732aba10e82232197e19fd84a79cbab6214c6 (patch) | |
| tree | c1b25c926d6a9ef48c647efd9167c621c297a20d | |
| parent | 5cf60f4482852220efa5f727b45617ee3b7ad773 (diff) | |
| download | kafka-python-1cb732aba10e82232197e19fd84a79cbab6214c6.tar.gz | |
pylint fixes
- temporarily ignore kafka/consumer/group.py
- temporarily ignore test/test_protocol.py
- disable-msg deprecated; use disable= instead
| -rw-r--r-- | kafka/consumer/group.py | 1 | ||||
| -rw-r--r-- | kafka/consumer/kafka.py | 2 | ||||
| -rw-r--r-- | kafka/consumer/simple.py | 4 | ||||
| -rw-r--r-- | kafka/protocol/struct.py | 2 | ||||
| -rw-r--r-- | test/fixtures.py | 2 | ||||
| -rw-r--r-- | test/test_protocol.py | 1 |
6 files changed, 7 insertions, 5 deletions
diff --git a/kafka/consumer/group.py b/kafka/consumer/group.py index 5d91469..dba5f60 100644 --- a/kafka/consumer/group.py +++ b/kafka/consumer/group.py @@ -1,3 +1,4 @@ +#pylint: skip-file from __future__ import absolute_import from collections import namedtuple diff --git a/kafka/consumer/kafka.py b/kafka/consumer/kafka.py index cdf8760..3f14444 100644 --- a/kafka/consumer/kafka.py +++ b/kafka/consumer/kafka.py @@ -664,7 +664,7 @@ class KafkaConsumer(object): # Otherwise we should re-raise the upstream exception # b/c it typically includes additional data about # the request that triggered it, and we do not want to drop that - raise # pylint: disable-msg=E0704 + raise # pylint: disable=E0704 (offset, ) = self.get_partition_offsets(topic, partition, request_time_ms, max_num_offsets=1) diff --git a/kafka/consumer/simple.py b/kafka/consumer/simple.py index 9e76730..9c2812b 100644 --- a/kafka/consumer/simple.py +++ b/kafka/consumer/simple.py @@ -1,7 +1,7 @@ from __future__ import absolute_import try: - from itertools import zip_longest as izip_longest, repeat # pylint: disable-msg=E0611 + from itertools import zip_longest as izip_longest, repeat # pylint: disable=E0611 except ImportError: from itertools import izip_longest as izip_longest, repeat # python 2 import logging @@ -167,7 +167,7 @@ class SimpleConsumer(Consumer): # Otherwise we should re-raise the upstream exception # b/c it typically includes additional data about # the request that triggered it, and we do not want to drop that - raise # pylint: disable-msg=E0704 + raise # pylint: disable=E0704 # send_offset_request log.info('Resetting topic-partition offset to %s for %s:%d', diff --git a/kafka/protocol/struct.py b/kafka/protocol/struct.py index 5b4c312..d340abf 100644 --- a/kafka/protocol/struct.py +++ b/kafka/protocol/struct.py @@ -21,7 +21,7 @@ class Struct(AbstractType): self.encode = self._encode_self @classmethod - def encode(cls, item): # pylint: disable-msg=E0202 + def encode(cls, item): # pylint: disable=E0202 bits = [] for i, field in enumerate(cls.SCHEMA.fields): bits.append(field.encode(item[i])) diff --git a/test/fixtures.py b/test/fixtures.py index 0ae1c1e..91a67c1 100644 --- a/test/fixtures.py +++ b/test/fixtures.py @@ -8,7 +8,7 @@ import time from six.moves import urllib import uuid -from six.moves.urllib.parse import urlparse # pylint: disable-msg=E0611,F0401 +from six.moves.urllib.parse import urlparse # pylint: disable=E0611,F0401 from test.service import ExternalService, SpawnedService from test.testutil import get_open_port diff --git a/test/test_protocol.py b/test/test_protocol.py index 7dfd44e..6c79829 100644 --- a/test/test_protocol.py +++ b/test/test_protocol.py @@ -1,3 +1,4 @@ +#pylint: skip-file from contextlib import contextmanager import struct |