mirror of
https://github.com/django/django.git
synced 2025-07-28 15:44:20 +00:00
Fixed #2705: added a select_for_update()
clause to querysets.
A number of people worked on this patch over the years -- Hawkeye, Colin Grady, KBS, sakyamuni, anih, jdemoor, and Issak Kelly. Thanks to them all, and apologies if I missed anyone. Special thanks to Dan Fairs for picking it up again at the end and seeing this through to commit. git-svn-id: http://code.djangoproject.com/svn/django/trunk@16058 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
parent
99c1794427
commit
8f0f73c7b8
14 changed files with 375 additions and 0 deletions
|
@ -1,11 +1,13 @@
|
|||
from django.core.exceptions import FieldError
|
||||
from django.db import connections
|
||||
from django.db import transaction
|
||||
from django.db.backends.util import truncate_name
|
||||
from django.db.models.sql.constants import *
|
||||
from django.db.models.sql.datastructures import EmptyResultSet
|
||||
from django.db.models.sql.expressions import SQLEvaluator
|
||||
from django.db.models.sql.query import get_proxied_model, get_order_dir, \
|
||||
select_related_descend, Query
|
||||
from django.db.utils import DatabaseError
|
||||
|
||||
class SQLCompiler(object):
|
||||
def __init__(self, query, connection, using):
|
||||
|
@ -117,6 +119,14 @@ class SQLCompiler(object):
|
|||
result.append('LIMIT %d' % val)
|
||||
result.append('OFFSET %d' % self.query.low_mark)
|
||||
|
||||
if self.query.select_for_update and self.connection.features.has_select_for_update:
|
||||
# If we've been asked for a NOWAIT query but the backend does not support it,
|
||||
# raise a DatabaseError otherwise we could get an unexpected deadlock.
|
||||
nowait = self.query.select_for_update_nowait
|
||||
if nowait and not self.connection.features.has_select_for_update_nowait:
|
||||
raise DatabaseError('NOWAIT is not supported on this database backend.')
|
||||
result.append(self.connection.ops.for_update_sql(nowait=nowait))
|
||||
|
||||
return ' '.join(result), tuple(params)
|
||||
|
||||
def as_nested_sql(self):
|
||||
|
@ -677,6 +687,11 @@ class SQLCompiler(object):
|
|||
resolve_columns = hasattr(self, 'resolve_columns')
|
||||
fields = None
|
||||
has_aggregate_select = bool(self.query.aggregate_select)
|
||||
# Set transaction dirty if we're using SELECT FOR UPDATE to ensure
|
||||
# a subsequent commit/rollback is executed, so any database locks
|
||||
# are released.
|
||||
if self.query.select_for_update and transaction.is_managed(self.using):
|
||||
transaction.set_dirty(self.using)
|
||||
for rows in self.execute_sql(MULTI):
|
||||
for row in rows:
|
||||
if resolve_columns:
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue