Merge pull request #951 from caullla/master

fix: speed up get deleted. remove unnecessary subquery
etianen · Nov 7, 2023 · c6321d6 · c6321d6
2 parents c6ed0c3 + d27bad2
commit c6321d6
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/reversion/models.py b/reversion/models.py
@@ -168,12 +168,11 @@ def get_deleted(self, model, model_db=None):
             # conditional expressions are being supported since django 3.0
             # DISTINCT ON works only for Postgres DB
             if connection.vendor == "postgresql" and django.VERSION >= (3, 0):
-                subquery = (
+                return (
                     self.get_for_model(model, model_db=model_db)
                     .filter(~models.Exists(model_qs))
                     .order_by("object_id", "-pk")
                     .distinct("object_id")
-                    .values("pk")
                 )
             else:
                 subquery = (