Fix various pylint errors

diff --git a/python/londiste/handlers/dispatch.py b/python/londiste/handlers/dispatch.py
index b2663448e4ca8ec125da6a304c89e007ce2f14c3..5916eee78ad3d895bb6690a1c5b063f9d30a5a7f 100644 (file)
--- a/python/londiste/handlers/dispatch.py
+++ b/python/londiste/handlers/dispatch.py
@@ -862,7 +862,6 @@ class Dispatcher(BaseHandler):
         copyed
         """
         _src_cols = _dst_cols = column_list
-        _write_hook = None
         condition = ' and '.join(cond_list)
 
         if self.conf.skip_fields:
@@ -877,6 +876,8 @@ class Dispatcher(BaseHandler):
         if self.encoding_validator:
             def _write_hook(obj, data):
                 return self.encoding_validator.validate_copy(data, _src_cols)
+        else:
+            _write_hook = None
 
         return skytools.full_copy(tablename, src_curs, dst_curs, _src_cols,
                                   condition, self.table_name, _dst_cols,

diff --git a/python/londiste/handlers/part.py b/python/londiste/handlers/part.py
index 784162f52ac26b9ec6de1f5004cfb66f700334a0..fa5ccb7787af67ed83d1b825012192cc3414787a 100644 (file)
--- a/python/londiste/handlers/part.py
+++ b/python/londiste/handlers/part.py
@@ -66,7 +66,7 @@ class PartHandler(TableHandler):
         curs.execute(q)
         self.local_part, self.max_part = curs.fetchone()
         if self.local_part is None or self.max_part is None:
-            raise Exeption('Error loading part info')
+            raise Exception('Error loading part info')
 
 # register handler class
 __londiste_handlers__ = [PartHandler]

diff --git a/python/londiste/handlers/qtable.py b/python/londiste/handlers/qtable.py
index 97b0b976b95bd0684adce156ca775f49d24a3c10..443d2496803cbf5ab12bc68d161bd90bea5e22b3 100644 (file)
--- a/python/londiste/handlers/qtable.py
+++ b/python/londiste/handlers/qtable.py
@@ -14,6 +14,8 @@ qsplitter  -  dummy handler to setup queue tables. All events are ignored. Table
 
 from londiste.handler import BaseHandler
 
+import pgq
+
 __all__ = ['QueueTableHandler', 'FakeLocalHandler', 'QueueSplitterHandler']