]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
just a pep8 pass of lib/sqlalchemy/engine/
authorDiana Clarke <diana.joan.clarke@gmail.com>
Mon, 19 Nov 2012 17:52:23 +0000 (12:52 -0500)
committerDiana Clarke <diana.joan.clarke@gmail.com>
Mon, 19 Nov 2012 17:52:23 +0000 (12:52 -0500)
lib/sqlalchemy/engine/__init__.py
lib/sqlalchemy/engine/base.py
lib/sqlalchemy/engine/ddl.py
lib/sqlalchemy/engine/default.py
lib/sqlalchemy/engine/interfaces.py
lib/sqlalchemy/engine/reflection.py
lib/sqlalchemy/engine/result.py
lib/sqlalchemy/engine/strategies.py
lib/sqlalchemy/engine/threadlocal.py
lib/sqlalchemy/engine/url.py
lib/sqlalchemy/engine/util.py

index f02e79f33178923de88ca6e37afffad5350eb718..5ef190394f696c13a0c9a8b73627774ab86dea97 100644 (file)
@@ -87,6 +87,7 @@ from . import util, strategies
 
 default_strategy = 'plain'
 
+
 def create_engine(*args, **kwargs):
     """Create a new :class:`.Engine` instance.
 
@@ -253,13 +254,13 @@ def create_engine(*args, **kwargs):
         opened above and beyond the pool_size setting, which defaults
         to five. this is only used with :class:`~sqlalchemy.pool.QueuePool`.
 
-    :param module=None: reference to a Python module object (the module itself, not
-        its string name).  Specifies an alternate DBAPI module to be used
-        by the engine's dialect.  Each sub-dialect references a specific DBAPI which
-        will be imported before first connect.  This parameter causes the
-        import to be bypassed, and the given module to be used instead.
-        Can be used for testing of DBAPIs as well as to inject "mock"
-        DBAPI implementations into the :class:`.Engine`.
+    :param module=None: reference to a Python module object (the module
+        itself, not its string name).  Specifies an alternate DBAPI module to
+        be used by the engine's dialect.  Each sub-dialect references a
+        specific DBAPI which will be imported before first connect.  This
+        parameter causes the import to be bypassed, and the given module to
+        be used instead. Can be used for testing of DBAPIs as well as to
+        inject "mock" DBAPI implementations into the :class:`.Engine`.
 
     :param pool=None: an already-constructed instance of
         :class:`~sqlalchemy.pool.Pool`, such as a
@@ -353,5 +354,3 @@ __all__ = (
     'create_engine',
     'engine_from_config',
     )
-
-
index 15bfe03cb3b605b1dcf2b68b4ecda943ebd7bc89..76946af3486e1625cb9c3fe3f4af417123c3b554 100644 (file)
@@ -17,6 +17,7 @@ from .interfaces import Connectable, Compiled
 from .util import _distill_params
 import contextlib
 
+
 class Connection(Connectable):
     """Provides high-level functionality for a wrapped DB-API connection.
 
@@ -118,10 +119,11 @@ class Connection(Connectable):
             result = connection.execution_options(stream_results=True).\\
                                 execute(stmt)
 
-        Note that any key/value can be passed to :meth:`.Connection.execution_options`,
-        and it will be stored in the ``_execution_options`` dictionary of
-        the :class:`.Connnection`.   It is suitable for usage by end-user
-        schemes to communicate with event listeners, for example.
+        Note that any key/value can be passed to
+        :meth:`.Connection.execution_options`, and it will be stored in the
+        ``_execution_options`` dictionary of the :class:`.Connnection`.   It
+        is suitable for usage by end-user schemes to communicate with
+        event listeners, for example.
 
         The keywords that are currently recognized by SQLAlchemy itself
         include all those listed under :meth:`.Executable.execution_options`,
@@ -328,7 +330,6 @@ class Connection(Connectable):
         del self.__connection
         self.__invalid = True
 
-
     def detach(self):
         """Detach the underlying DB-API connection from its connection pool.
 
@@ -377,7 +378,8 @@ class Connection(Connectable):
 
         :meth:`.Connection.begin_twophase` - use a two phase /XID transaction
 
-        :meth:`.Engine.begin` - context manager available from :class:`.Engine`.
+        :meth:`.Engine.begin` - context manager available from
+        :class:`.Engine`.
 
         """
 
@@ -414,8 +416,8 @@ class Connection(Connectable):
 
         The returned object is an instance of :class:`.TwoPhaseTransaction`,
         which in addition to the methods provided by
-        :class:`.Transaction`, also provides a :meth:`~.TwoPhaseTransaction.prepare`
-        method.
+        :class:`.Transaction`, also provides a
+        :meth:`~.TwoPhaseTransaction.prepare` method.
 
         :param xid: the two phase transaction id.  If not supplied, a
           random id will be generated.
@@ -593,7 +595,8 @@ class Connection(Connectable):
         return self.execute(object, *multiparams, **params).scalar()
 
     def execute(self, object, *multiparams, **params):
-        """Executes the a SQL statement construct and returns a :class:`.ResultProxy`.
+        """Executes the a SQL statement construct and returns a
+        :class:`.ResultProxy`.
 
         :param object: The statement to be executed.  May be
          one of:
@@ -662,7 +665,6 @@ class Connection(Connectable):
                                 "Unexecutable object type: %s" %
                                 type(object))
 
-
     def _execute_function(self, func, multiparams, params):
         """Execute a sql.FunctionElement object."""
 
@@ -752,7 +754,6 @@ class Connection(Connectable):
                             dialect=dialect, column_keys=keys,
                             inline=len(distilled_params) > 1)
 
-
         ret = self._execute_context(
             dialect,
             dialect.execution_ctx_cls._init_compiled,
@@ -875,7 +876,6 @@ class Connection(Connectable):
                                 context)
             raise
 
-
         if self._has_events:
             self.dispatch.after_cursor_execute(self, cursor,
                                                 statement,
@@ -1005,8 +1005,7 @@ class Connection(Connectable):
                 context.handle_dbapi_exception(e)
 
             is_disconnect = isinstance(e, self.dialect.dbapi.Error) and \
-                                self.dialect.is_disconnect(e, self.__connection, cursor)
-
+                self.dialect.is_disconnect(e, self.__connection, cursor)
 
             if is_disconnect:
                 dbapi_conn_wrapper = self.connection
@@ -1055,7 +1054,6 @@ class Connection(Connectable):
         basestring: _execute_text
     }
 
-
     def default_schema_name(self):
         return self.engine.dialect.get_default_schema_name(self)
 
@@ -1219,6 +1217,7 @@ class Transaction(object):
         else:
             self.rollback()
 
+
 class RootTransaction(Transaction):
     def __init__(self, connection):
         super(RootTransaction, self).__init__(connection, None)
@@ -1383,10 +1382,10 @@ class Engine(Connectable, log.Identified):
             shard1 = primary_engine.execution_options(shard_id="shard1")
             shard2 = primary_engine.execution_options(shard_id="shard2")
 
-        Above, the ``shard1`` engine serves as a factory for :class:`.Connection`
-        objects that will contain the execution option ``shard_id=shard1``,
-        and ``shard2`` will produce :class:`.Connection` objects that contain
-        the execution option ``shard_id=shard2``.
+        Above, the ``shard1`` engine serves as a factory for
+        :class:`.Connection` objects that will contain the execution option
+        ``shard_id=shard1``, and ``shard2`` will produce :class:`.Connection`
+        objects that contain the execution option ``shard_id=shard2``.
 
         An event handler can consume the above execution option to perform
         a schema switch or other operation, given a connection.  Below
@@ -1467,7 +1466,6 @@ class Engine(Connectable, log.Identified):
         """
         self.pool = self.pool._replace()
 
-
     def _execute_default(self, default):
         with self.contextual_connect() as conn:
             return conn._execute_default(default, (), {})
@@ -1502,7 +1500,6 @@ class Engine(Connectable, log.Identified):
             if not self.close_with_result:
                 self.conn.close()
 
-
     def begin(self, close_with_result=False):
         """Return a context manager delivering a :class:`.Connection`
         with a :class:`.Transaction` established.
@@ -1519,11 +1516,11 @@ class Engine(Connectable, log.Identified):
 
         The ``close_with_result`` flag is normally ``False``, and indicates
         that the :class:`.Connection` will be closed when the operation
-        is complete.   When set to ``True``, it indicates the :class:`.Connection`
-        is in "single use" mode, where the :class:`.ResultProxy`
-        returned by the first call to :meth:`.Connection.execute` will
-        close the :class:`.Connection` when that :class:`.ResultProxy`
-        has exhausted all result rows.
+        is complete.   When set to ``True``, it indicates the
+        :class:`.Connection` is in "single use" mode, where the
+        :class:`.ResultProxy` returned by the first call to
+        :meth:`.Connection.execute` will close the :class:`.Connection` when
+        that :class:`.ResultProxy` has exhausted all result rows.
 
         .. versionadded:: 0.7.6
 
@@ -1635,29 +1632,33 @@ class Engine(Connectable, log.Identified):
     def connect(self, **kwargs):
         """Return a new :class:`.Connection` object.
 
-        The :class:`.Connection` object is a facade that uses a DBAPI connection internally
-        in order to communicate with the database.  This connection is procured
-        from the connection-holding :class:`.Pool` referenced by this :class:`.Engine`.
-        When the :meth:`~.Connection.close` method of the :class:`.Connection` object is called,
-        the underlying DBAPI connection is then returned to the connection pool,
-        where it may be used again in a subsequent call to :meth:`~.Engine.connect`.
+        The :class:`.Connection` object is a facade that uses a DBAPI
+        connection internally in order to communicate with the database.  This
+        connection is procured from the connection-holding :class:`.Pool`
+        referenced by this :class:`.Engine`. When the
+        :meth:`~.Connection.close` method of the :class:`.Connection` object
+        is called, the underlying DBAPI connection is then returned to the
+        connection pool, where it may be used again in a subsequent call to
+        :meth:`~.Engine.connect`.
 
         """
 
         return self._connection_cls(self, **kwargs)
 
     def contextual_connect(self, close_with_result=False, **kwargs):
-        """Return a :class:`.Connection` object which may be part of some ongoing context.
+        """Return a :class:`.Connection` object which may be part of some
+        ongoing context.
 
         By default, this method does the same thing as :meth:`.Engine.connect`.
         Subclasses of :class:`.Engine` may override this method
         to provide contextual behavior.
 
-        :param close_with_result: When True, the first :class:`.ResultProxy` created
-          by the :class:`.Connection` will call the :meth:`.Connection.close` method
-          of that connection as soon as any pending result rows are exhausted.
-          This is used to supply the "connectionless execution" behavior provided
-          by the :meth:`.Engine.execute` method.
+        :param close_with_result: When True, the first :class:`.ResultProxy`
+          created by the :class:`.Connection` will call the
+          :meth:`.Connection.close` method of that connection as soon as any
+          pending result rows are exhausted. This is used to supply the
+          "connectionless execution" behavior provided by the
+          :meth:`.Engine.execute` method.
 
         """
 
@@ -1702,6 +1703,7 @@ class Engine(Connectable, log.Identified):
 
         return self.pool.unique_connection()
 
+
 class OptionEngine(Engine):
     def __init__(self, proxied, execution_options):
         self._proxied = proxied
index f95e549b086cab93137879c690294eeb03cc2adf..8da678525d10c85adb80632e30b2b140b4ac8070 100644 (file)
@@ -14,8 +14,11 @@ class DDLBase(schema.SchemaVisitor):
     def __init__(self, connection):
         self.connection = connection
 
+
 class SchemaGenerator(DDLBase):
-    def __init__(self, dialect, connection, checkfirst=False, tables=None, **kwargs):
+
+    def __init__(self, dialect, connection, checkfirst=False,
+                 tables=None, **kwargs):
         super(SchemaGenerator, self).__init__(connection, **kwargs)
         self.checkfirst = checkfirst
         self.tables = tables and set(tables) or None
@@ -103,7 +106,9 @@ class SchemaGenerator(DDLBase):
 
 
 class SchemaDropper(DDLBase):
-    def __init__(self, dialect, connection, checkfirst=False, tables=None, **kwargs):
+
+    def __init__(self, dialect, connection, checkfirst=False,
+                 tables=None, **kwargs):
         super(SchemaDropper, self).__init__(connection, **kwargs)
         self.checkfirst = checkfirst
         self.tables = tables
@@ -116,15 +121,22 @@ class SchemaDropper(DDLBase):
             tables = self.tables
         else:
             tables = metadata.tables.values()
-        collection = [t for t in reversed(sql_util.sort_tables(tables))
-                                if self._can_drop_table(t)]
-        seq_coll = [s for s in metadata._sequences.values()
-                                if s.column is None and self._can_drop_sequence(s)]
 
-        metadata.dispatch.before_drop(metadata, self.connection,
-                                            tables=collection,
-                                            checkfirst=self.checkfirst,
-                                            _ddl_runner=self)
+        collection = [
+            t
+            for t in reversed(sql_util.sort_tables(tables))
+            if self._can_drop_table(t)
+        ]
+
+        seq_coll = [
+            s
+            for s in metadata._sequences.values()
+            if s.column is None and self._can_drop_sequence(s)
+        ]
+
+        metadata.dispatch.before_drop(
+            metadata, self.connection, tables=collection,
+            checkfirst=self.checkfirst, _ddl_runner=self)
 
         for table in collection:
             self.traverse_single(table, drop_ok=True)
@@ -132,10 +144,9 @@ class SchemaDropper(DDLBase):
         for seq in seq_coll:
             self.traverse_single(seq, drop_ok=True)
 
-        metadata.dispatch.after_drop(metadata, self.connection,
-                                            tables=collection,
-                                            checkfirst=self.checkfirst,
-                                            _ddl_runner=self)
+        metadata.dispatch.after_drop(
+            metadata, self.connection, tables=collection,
+            checkfirst=self.checkfirst, _ddl_runner=self)
 
     def _can_drop_table(self, table):
         self.dialect.validate_identifier(table.name)
index c536d1a4cd54421a5f754f875561bd69b4910fc8..335515a7b337aa26cfb48da77f659ee3de605e65 100644 (file)
@@ -45,7 +45,6 @@ class DefaultDialect(interfaces.Dialect):
     postfetch_lastrowid = True
     implicit_returning = False
 
-
     supports_native_enum = False
     supports_native_boolean = False
 
@@ -66,7 +65,6 @@ class DefaultDialect(interfaces.Dialect):
     description_encoding = 'use_encoding'
     # end Py2K
 
-
     name = 'default'
 
     # length at which to truncate
@@ -341,6 +339,7 @@ class DefaultDialect(interfaces.Dialect):
         # the configured default of this dialect.
         self.set_isolation_level(dbapi_conn, self.default_isolation_level)
 
+
 class DefaultExecutionContext(interfaces.ExecutionContext):
     isinsert = False
     isupdate = False
index 6d0b1cb611ef42e05e0c9610a7f4e4fc5805beca..66856a00ecaa24d4f5b1bb55a87742cf59310edf 100644 (file)
@@ -8,6 +8,7 @@
 
 from .. import util, event, events
 
+
 class Dialect(object):
     """Define the behavior of a specific database and DB-API combination.
 
@@ -485,7 +486,8 @@ class Dialect(object):
 
         raise NotImplementedError()
 
-    def do_execute_no_params(self, cursor, statement, parameters, context=None):
+    def do_execute_no_params(self, cursor, statement, parameters,
+                             context=None):
         """Provide an implementation of ``cursor.execute(statement)``.
 
         The parameter collection should not be sent.
@@ -767,8 +769,8 @@ class TypeCompiler(object):
 class Connectable(object):
     """Interface for an object which supports execution of SQL constructs.
 
-    The two implementations of :class:`.Connectable` are :class:`.Connection` and
-    :class:`.Engine`.
+    The two implementations of :class:`.Connectable` are
+    :class:`.Connection` and :class:`.Engine`.
 
     Connectable must also implement the 'dialect' member which references a
     :class:`.Dialect` instance.
@@ -777,7 +779,6 @@ class Connectable(object):
 
     dispatch = event.dispatcher(events.ConnectionEvents)
 
-
     def connect(self, **kwargs):
         """Return a :class:`.Connection` object.
 
@@ -801,17 +802,19 @@ class Connectable(object):
 
         raise NotImplementedError()
 
-    @util.deprecated("0.7", "Use the create() method on the given schema "
-                            "object directly, i.e. :meth:`.Table.create`, "
-                            ":meth:`.Index.create`, :meth:`.MetaData.create_all`")
+    @util.deprecated("0.7",
+                     "Use the create() method on the given schema "
+                     "object directly, i.e. :meth:`.Table.create`, "
+                     ":meth:`.Index.create`, :meth:`.MetaData.create_all`")
     def create(self, entity, **kwargs):
         """Emit CREATE statements for the given schema entity."""
 
         raise NotImplementedError()
 
-    @util.deprecated("0.7", "Use the drop() method on the given schema "
-                            "object directly, i.e. :meth:`.Table.drop`, "
-                            ":meth:`.Index.drop`, :meth:`.MetaData.drop_all`")
+    @util.deprecated("0.7",
+                     "Use the drop() method on the given schema "
+                     "object directly, i.e. :meth:`.Table.drop`, "
+                     ":meth:`.Index.drop`, :meth:`.MetaData.drop_all`")
     def drop(self, entity, **kwargs):
         """Emit DROP statements for the given schema entity."""
 
@@ -834,4 +837,3 @@ class Connectable(object):
 
     def _execute_clauseelement(self, elem, multiparams=None, params=None):
         raise NotImplementedError()
-
index 4505aa18a1332e82d3fbb2b435fde76b8ebb6135..8367d87614d30095f9519db3fab25ad4227bffe9 100644 (file)
@@ -33,6 +33,7 @@ from ..util import topological
 from .. import inspection
 from .base import Connectable
 
+
 @util.decorator
 def cache(fn, self, con, *args, **kw):
     info_cache = kw.get('info_cache', None)
@@ -107,17 +108,19 @@ class Inspector(object):
 
     @classmethod
     def from_engine(cls, bind):
-        """Construct a new dialect-specific Inspector object from the given engine or connection.
+        """Construct a new dialect-specific Inspector object from the given
+        engine or connection.
 
         :param bind: a :class:`~sqlalchemy.engine.base.Connectable`,
           which is typically an instance of
           :class:`~sqlalchemy.engine.Engine` or
           :class:`~sqlalchemy.engine.Connection`.
 
-        This method differs from direct a direct constructor call of :class:`.Inspector`
-        in that the :class:`~sqlalchemy.engine.base.Dialect` is given a chance to provide
-        a dialect-specific :class:`.Inspector` instance, which may provide additional
-        methods.
+        This method differs from direct a direct constructor call of
+        :class:`.Inspector` in that the
+        :class:`~sqlalchemy.engine.base.Dialect` is given a chance to provide
+        a dialect-specific :class:`.Inspector` instance, which may provide
+        additional methods.
 
         See the example at :class:`.Inspector`.
 
@@ -194,15 +197,16 @@ class Inspector(object):
         return tnames
 
     def get_table_options(self, table_name, schema=None, **kw):
-        """Return a dictionary of options specified when the table of the given name was created.
+        """Return a dictionary of options specified when the table of the
+        given name was created.
 
         This currently includes some options that apply to MySQL tables.
 
         """
         if hasattr(self.dialect, 'get_table_options'):
-            return self.dialect.get_table_options(self.bind, table_name, schema,
-                                                  info_cache=self.info_cache,
-                                                  **kw)
+            return self.dialect.get_table_options(
+                self.bind, table_name, schema,
+                info_cache=self.info_cache, **kw)
         return {}
 
     def get_view_names(self, schema=None):
@@ -285,7 +289,6 @@ class Inspector(object):
                                               info_cache=self.info_cache,
                                               **kw)
 
-
     def get_foreign_keys(self, table_name, schema=None, **kw):
         """Return information about foreign_keys in `table_name`.
 
@@ -341,7 +344,8 @@ class Inspector(object):
                                             info_cache=self.info_cache, **kw)
 
     def reflecttable(self, table, include_columns, exclude_columns=()):
-        """Given a Table object, load its internal constructs based on introspection.
+        """Given a Table object, load its internal constructs based on
+        introspection.
 
         This is the underlying method used by most dialects to produce
         table reflection.  Direct usage is like::
@@ -410,8 +414,9 @@ class Inspector(object):
 
             colargs = []
             if col_d.get('default') is not None:
-                # the "default" value is assumed to be a literal SQL expression,
-                # so is wrapped in text() so that no quoting occurs on re-issuance.
+                # the "default" value is assumed to be a literal SQL
+                # expression, so is wrapped in text() so that no quoting
+                # occurs on re-issuance.
                 colargs.append(
                     sa_schema.DefaultClause(
                         sql.text(col_d['default']), _reflected=True
@@ -437,11 +442,18 @@ class Inspector(object):
         # Primary keys
         pk_cons = self.get_pk_constraint(table_name, schema, **tblkw)
         if pk_cons:
-            pk_cols = [table.c[pk]
-                        for pk in pk_cons['constrained_columns']
-                        if pk in table.c and pk not in exclude_columns
-                    ] + [pk for pk in table.primary_key if pk.key in exclude_columns]
-            primary_key_constraint = sa_schema.PrimaryKeyConstraint(name=pk_cons.get('name'),
+            pk_cols = [
+                table.c[pk]
+                for pk in pk_cons['constrained_columns']
+                if pk in table.c and pk not in exclude_columns
+            ]
+            pk_cols += [
+                pk
+                for pk in table.primary_key
+                if pk.key in exclude_columns
+            ]
+            primary_key_constraint = sa_schema.PrimaryKeyConstraint(
+                name=pk_cons.get('name'),
                 *pk_cols
             )
 
index 6962a4d1e2dc3fe6e15ecc406adc2fc960241921..98b0ea4b236febc9f9f32f37aacc8fb12dad49de 100644 (file)
@@ -831,6 +831,7 @@ class ResultProxy(object):
         else:
             return None
 
+
 class BufferedRowResultProxy(ResultProxy):
     """A ResultProxy with row buffering behavior.
 
@@ -934,6 +935,7 @@ class FullyBufferedResultProxy(ResultProxy):
         self.__rowbuffer = collections.deque()
         return ret
 
+
 class BufferedColumnRow(RowProxy):
     def __init__(self, parent, row, processors, keymap):
         # preprocess row
index 1e321603ed92f75ede7bd89fc0ff062404165713..2db1bfcc56bef32c2f23cfd1311640f7ac42b8bf 100644 (file)
@@ -79,19 +79,18 @@ class DefaultEngineStrategy(EngineStrategy):
                 try:
                     return dialect.connect(*cargs, **cparams)
                 except Exception, e:
+                    invalidated = dialect.is_disconnect(e, None, None)
                     # Py3K
                     #raise exc.DBAPIError.instance(None, None,
-                    #                   e, dialect.dbapi.Error,
-                    #                   connection_invalidated=
-                    #                       dialect.is_disconnect(e, None, None)
-                    #                       ) from e
+                    #    e, dialect.dbapi.Error,
+                    #    connection_invalidated=invalidated
+                    #) from e
                     # Py2K
                     import sys
                     raise exc.DBAPIError.instance(
-                                None, None, e, dialect.dbapi.Error,
-                                connection_invalidated=
-                                        dialect.is_disconnect(e, None, None)), \
-                                None, sys.exc_info()[2]
+                        None, None, e, dialect.dbapi.Error,
+                        connection_invalidated=invalidated
+                    ), None, sys.exc_info()[2]
                     # end Py2K
 
             creator = kwargs.pop('creator', connect)
@@ -107,9 +106,9 @@ class DefaultEngineStrategy(EngineStrategy):
                          'echo': 'echo_pool',
                          'timeout': 'pool_timeout',
                          'recycle': 'pool_recycle',
-                         'events':'pool_events',
-                         'use_threadlocal':'pool_threadlocal',
-                         'reset_on_return':'pool_reset_on_return'}
+                         'events': 'pool_events',
+                         'use_threadlocal': 'pool_threadlocal',
+                         'reset_on_return': 'pool_reset_on_return'}
             for k in util.get_cls_kwargs(poolclass):
                 tk = translate.get(k, k)
                 if tk in kwargs:
@@ -147,7 +146,8 @@ class DefaultEngineStrategy(EngineStrategy):
             do_on_connect = dialect.on_connect()
             if do_on_connect:
                 def on_connect(dbapi_connection, connection_record):
-                    conn = getattr(dbapi_connection, '_sqla_unwrap', dbapi_connection)
+                    conn = getattr(
+                        dbapi_connection, '_sqla_unwrap', dbapi_connection)
                     if conn is None:
                         return
                     do_on_connect(conn)
@@ -238,12 +238,14 @@ class MockEngineStrategy(EngineStrategy):
             kwargs['checkfirst'] = False
             from sqlalchemy.engine import ddl
 
-            ddl.SchemaGenerator(self.dialect, self, **kwargs).traverse_single(entity)
+            ddl.SchemaGenerator(
+                self.dialect, self, **kwargs).traverse_single(entity)
 
         def drop(self, entity, **kwargs):
             kwargs['checkfirst'] = False
             from sqlalchemy.engine import ddl
-            ddl.SchemaDropper(self.dialect, self, **kwargs).traverse_single(entity)
+            ddl.SchemaDropper(
+                self.dialect, self, **kwargs).traverse_single(entity)
 
         def _run_visitor(self, visitorcallable, element,
                                         connection=None,
index 2ff498db5fa3623848980470c553219296efae30..06246e854dc71c34694d5322fc3bae452765abb3 100644 (file)
@@ -6,16 +6,19 @@
 
 """Provides a thread-local transactional wrapper around the root Engine class.
 
-The ``threadlocal`` module is invoked when using the ``strategy="threadlocal"`` flag
-with :func:`~sqlalchemy.engine.create_engine`.  This module is semi-private and is
-invoked automatically when the threadlocal engine strategy is used.
+The ``threadlocal`` module is invoked when using the
+``strategy="threadlocal"`` flag with :func:`~sqlalchemy.engine.create_engine`.
+This module is semi-private and is invoked automatically when the threadlocal
+engine strategy is used.
 """
 
-from .. import util, event
+from .. import util
 from . import base
 import weakref
 
+
 class TLConnection(base.Connection):
+
     def __init__(self, *arg, **kw):
         super(TLConnection, self).__init__(*arg, **kw)
         self.__opencount = 0
@@ -33,16 +36,18 @@ class TLConnection(base.Connection):
         self.__opencount = 0
         base.Connection.close(self)
 
+
 class TLEngine(base.Engine):
-    """An Engine that includes support for thread-local managed transactions."""
+    """An Engine that includes support for thread-local managed
+    transactions.
 
+    """
     _tl_connection_cls = TLConnection
 
     def __init__(self, *args, **kwargs):
         super(TLEngine, self).__init__(*args, **kwargs)
         self._connections = util.threading.local()
 
-
     def contextual_connect(self, **kw):
         if not hasattr(self._connections, 'conn'):
             connection = None
@@ -52,21 +57,24 @@ class TLEngine(base.Engine):
         if connection is None or connection.closed:
             # guards against pool-level reapers, if desired.
             # or not connection.connection.is_valid:
-            connection = self._tl_connection_cls(self, self.pool.connect(), **kw)
-            self._connections.conn = conn = weakref.ref(connection)
+            connection = self._tl_connection_cls(
+                self, self.pool.connect(), **kw)
+            self._connections.conn = weakref.ref(connection)
 
         return connection._increment_connect()
 
     def begin_twophase(self, xid=None):
         if not hasattr(self._connections, 'trans'):
             self._connections.trans = []
-        self._connections.trans.append(self.contextual_connect().begin_twophase(xid=xid))
+        self._connections.trans.append(
+            self.contextual_connect().begin_twophase(xid=xid))
         return self
 
     def begin_nested(self):
         if not hasattr(self._connections, 'trans'):
             self._connections.trans = []
-        self._connections.trans.append(self.contextual_connect().begin_nested())
+        self._connections.trans.append(
+            self.contextual_connect().begin_nested())
         return self
 
     def begin(self):
index 5947deec303d822618824a1bc0660c22621a70cb..42b5de7db2ddbbaae9bf228a438797e62b2081b3 100644 (file)
@@ -7,8 +7,9 @@
 """Provides the :class:`~sqlalchemy.engine.url.URL` class which encapsulates
 information about a database connection specification.
 
-The URL object is created automatically when :func:`~sqlalchemy.engine.create_engine` is called
-with a string argument; alternatively, the URL is a public-facing construct which can
+The URL object is created automatically when
+:func:`~sqlalchemy.engine.create_engine` is called with a string
+argument; alternatively, the URL is a public-facing construct which can
 be used directly and is also accepted directly by ``create_engine()``.
 """
 
@@ -124,8 +125,8 @@ class URL(object):
 
         :param \**kw: Optional, alternate key names for url attributes.
 
-        :param names: Deprecated.  Same purpose as the keyword-based alternate names,
-            but correlates the name to the original positionally.
+        :param names: Deprecated.  Same purpose as the keyword-based alternate
+            names, but correlates the name to the original positionally.
         """
 
         translated = {}
@@ -141,6 +142,7 @@ class URL(object):
                 translated[name] = getattr(self, sname)
         return translated
 
+
 def make_url(name_or_url):
     """Given a string or unicode instance, produce a new URL instance.
 
@@ -153,6 +155,7 @@ def make_url(name_or_url):
     else:
         return name_or_url
 
+
 def _parse_rfc1738_args(name):
     pattern = re.compile(r'''
             (?P<name>[\w\+]+)://
@@ -165,8 +168,7 @@ def _parse_rfc1738_args(name):
                 (?::(?P<port>[^/]*))?
             )?
             (?:/(?P<database>.*))?
-            '''
-            , re.X)
+            ''', re.X)
 
     m = pattern.match(name)
     if m is not None:
@@ -184,7 +186,8 @@ def _parse_rfc1738_args(name):
         components['query'] = query
 
         if components['password'] is not None:
-            components['password'] = urllib.unquote_plus(components['password'])
+            components['password'] = \
+                urllib.unquote_plus(components['password'])
 
         name = components.pop('name')
         return URL(name, **components)
@@ -192,11 +195,12 @@ def _parse_rfc1738_args(name):
         raise exc.ArgumentError(
             "Could not parse rfc1738 URL from string '%s'" % name)
 
+
 def _parse_keyvalue_args(name):
-    m = re.match( r'(\w+)://(.*)', name)
+    m = re.match(r'(\w+)://(.*)', name)
     if m is not None:
         (name, args) = m.group(1, 2)
-        opts = dict( util.parse_qsl( args ) )
+        opts = dict(util.parse_qsl(args))
         return URL(name, *opts)
     else:
         return None
index 6bf8f2d3fd92855e981c67e5edf604e9612e565a..8cdfd5cdf03cc11a1e5c36ff057afa2afdece543 100644 (file)
@@ -6,6 +6,7 @@
 
 from .. import util
 
+
 def _coerce_config(configuration, prefix):
     """Convert configuration values to expected types."""
 
@@ -26,6 +27,7 @@ def _coerce_config(configuration, prefix):
         util.coerce_kw_type(options, option, type_)
     return options
 
+
 def connection_memoize(key):
     """Decorator, memoize a function in a connection.info stash.
 
@@ -44,6 +46,7 @@ def connection_memoize(key):
 
     return decorated
 
+
 def py_fallback():
     def _distill_params(multiparams, params):
         """Given arguments from the calling form *multiparams, **params,
@@ -89,5 +92,3 @@ try:
     from sqlalchemy.cutils import _distill_params
 except ImportError:
     globals().update(py_fallback())
-
-