From d79d48ca55a3576723cd3bdde05b426de07c1e69 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Ga=C3=ABtan=20de=20Menten?= Date: Wed, 4 Nov 2009 13:19:47 +0000 Subject: [PATCH] Using generators for small lists is highly inefficient. This change shoves 6% of total time for large ORM queries. Not bad for a 2 characters change :) --- lib/sqlalchemy/orm/mapper.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/sqlalchemy/orm/mapper.py b/lib/sqlalchemy/orm/mapper.py index 6bc55c8315..146c4d6737 100644 --- a/lib/sqlalchemy/orm/mapper.py +++ b/lib/sqlalchemy/orm/mapper.py @@ -1583,7 +1583,7 @@ class Mapper(object): identity_class = self._identity_class def identity_key(row): - return (identity_class, tuple(row[column] for column in pk_cols)) + return (identity_class, tuple([row[column] for column in pk_cols])) new_populators = [] existing_populators = [] -- 2.47.3