Re: [Django] #36315: Use task groups/cancel scopes where possible

[Django]

#36315: Use task groups/cancel scopes where possible
-------------------------------------+-------------------------------------
Reporter: Thomas Grainger | Owner: Thomas
Type: | Grainger
Cleanup/optimization | Status: assigned
Component: HTTP handling | Version: dev
Severity: Normal | Resolution:
Keywords: async | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Carlton Gibson):

* stage: Accepted => Ready for checkin

Comment:

I looked at both the PRs here, and they seem correct.
--
Ticket URL: <https://code.djangoproject.com/ticket/36315#comment:5>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

[Django]

#36315: Use task groups/cancel scopes where possible
-------------------------------------+-------------------------------------
Reporter: Thomas Grainger | Owner: Thomas
Type: | Grainger
Cleanup/optimization | Status: assigned
Component: HTTP handling | Version: dev
Severity: Normal | Resolution:
Keywords: async | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Jacob Walls <jacobtylerwalls@…>):

In [changeset:"2501958b5127020411df6271445ccfd0906df70e" 2501958]:
{{{#!CommitTicketReference repository=""
revision="2501958b5127020411df6271445ccfd0906df70e"
Refs #36315 -- Replaced manual task and cancellation handling with
TaskGroup in ASGIHandler.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36315#comment:7>

[Django]

#36315: Use task groups/cancel scopes where possible
-------------------------------------+-------------------------------------
Reporter: Thomas Grainger | Owner: Thomas
Type: | Grainger
Cleanup/optimization | Status: assigned
Component: HTTP handling | Version: dev
Severity: Normal | Resolution:
Keywords: async | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Jacob Walls <jacobtylerwalls@…>):

In [changeset:"796cf3d325b4a1b9d9b5361c2c8c28a5edcfe89b" 796cf3d3]:
{{{#!CommitTicketReference repository=""
revision="796cf3d325b4a1b9d9b5361c2c8c28a5edcfe89b"
Refs #36315 -- Used contextlib.closing() in ASGIHandler.handle().
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36315#comment:6>

[Django]

#36315: Use task groups/cancel scopes where possible
-------------------------------------+-------------------------------------
Reporter: Thomas Grainger | Owner: Thomas
Type: | Grainger

Cleanup/optimization | Status: closed

Component: HTTP handling | Version: dev

Severity: Normal | Resolution: fixed

Keywords: async | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by Jacob Walls <jacobtylerwalls@…>):

* resolution: => fixed
* status: assigned => closed

Comment:

In [changeset:"27687475265f88bc0a0bcbfe2ba26da306bbfc20" 2768747]:
{{{#!CommitTicketReference repository=""
revision="27687475265f88bc0a0bcbfe2ba26da306bbfc20"
Fixed #36315 -- Used TaskGroup instead of asyncio.gather().
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36315#comment:8>