Lock Receipt during validation and other concurrency improvements

[npazosmendez]

No description provided.

[WhyNotHugo]

Se ve bien, sólo tengo feedback sobre detalles menores.

[WhyNotHugo]

Podés reusar el QuerySet existente acá:

Suggested change

	qs = Receipt.objects.filter(id__in=ids, receipt_number__isnull=False).select_for_update()
	qs = qs.select_for_update()

[npazosmendez]

Eso intenté en un principio, pero me da esta excepción:

self = <django.db.backends.utils.CursorWrapper object at 0x7f219c6762a0>
sql = 'SELECT "afip_receipt"."id", "afip_receipt"."point_of_sales_id", "afip_receipt"."receipt_type_id", "afip_receipt"."con..."."id" = %s AND "afip_receiptvalidation"."id" IS NULL) ORDER BY "afip_receipt"."receipt_number" ASC LIMIT 1 FOR UPDATE'
params = (12,), ignored_wrapper_args = (False, {'connection': <DatabaseWrapper vendor='postgresql' alias='default'>, 'cursor': <django.db.backends.utils.CursorWrapper object at 0x7f219c6762a0>})

    def _execute(self, sql, params, *ignored_wrapper_args):
        # Raise a warning during app initialization (stored_app_configs is only
        # ever set during testing).
        if not apps.ready and not apps.stored_app_configs:
            warnings.warn(self.APPS_NOT_READY_WARNING_MSG, category=RuntimeWarning)
        self.db.validate_no_broken_transaction()
        with self.db.wrap_database_errors:
            if params is None:
                # params default might be backend specific.
                return self.cursor.execute(sql)
            else:
>               return self.cursor.execute(sql, params)
E               django.db.utils.NotSupportedError: FOR UPDATE cannot be applied to the nullable side of an outer join

.tox/live/lib/python3.12/site-packages/django/db/backends/utils.py:105: NotSupportedError

Me parece que el problema es el qs = self.filter(validation__isnull=True) de un par de lineas más arriba. Por lo que leí, Postgres dice que no puede lockear las filas según un criterio como ese, porque en cualquier momento podría suceder que validation__isnull=True si se agregara un Validation que apunte al Receipt; tendría que lockear la tabla Validations entera supongo, o algo similar.

Decime si estoy tirando fruta, no sé mucho PG pero menos aun Django

[WhyNotHugo]

Idem