p31829507
/
django
mirror of https://github.com/django/django.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Refs #26064 -- Avoided unnecessary list slicing in migration optimizer. 

The in_between list is only necessary if an optimization is possible.
This commit is contained in:
Simon Charette 2020-03-02 02:42:06 -05:00 committed by GitHub
parent a49c2b6bf0
commit daaa894960
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
django/db/migrations/optimizer.py
View File

@ -45,9 +45,9 @@ class MigrationOptimizer:
right = True # Should we reduce on the right or on the left. right = True # Should we reduce on the right or on the left.
# Compare it to each operation after it # Compare it to each operation after it
for j, other in enumerate(operations[i + 1:]): for j, other in enumerate(operations[i + 1:]):
in_between = operations[i + 1:i + j + 1]
result = operation.reduce(other, app_label) result = operation.reduce(other, app_label)
if isinstance(result, list): if isinstance(result, list):
in_between = operations[i + 1:i + j + 1]
if right: if right:
new_operations.extend(in_between) new_operations.extend(in_between)
new_operations.extend(result) new_operations.extend(result)