#37125: Use __new__ to sanitize TaskResult instead of __post_init__ to half
memory
usage
--------------------------------------+------------------------------------
Reporter: Johannes Maron | Owner: zky
Type: Cleanup/optimization | Status: assigned
Component: Tasks | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
--------------------------------------+------------------------------------
Comment (by Johannes Maron):
@zky go for it! I think those changes are fairly conflict-free, though.
IMHO, the real work will be writing good tests to prevent performance
regression. The actual patch is probably just those four lines:
{{{
def __new__(cls, *args, **kwargs):
kwargs["args"] = normalize_json(kwargs["args"])
kwargs["kwargs"] = normalize_json(kwargs["kwargs"])
return super().__new__(cls)
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/37125#comment:5>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion visit
https://groups.google.com/d/msgid/django-updates/0107019e730ee80f-63b22e34-2111-44d6-8281-cb54022dd4c2-000000%40eu-central-1.amazonses.com.
