#60697 closed defect (bug) (fixed)
HTML API: Defer applying updates until necessary.
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.5 | Priority: | normal |
Severity: | normal | Version: | 6.3 |
Component: | HTML API | Keywords: | has-patch has-unit-tests dev-reviewed fixed-major |
Focuses: | Cc: |
Description
When making repeated updates to a document, the Tag Processor will end
up copying the entire document once for every update. This can lead to
catastrophic runtime behavior in the worse case.
However, when batch-applying updates it's able to copy chunks of the
document in one go and only end up copying the entire document once
for the entire batch.
Previously the Tag Processor has been eagerly applying updates, but in
this patch it defers applying those updates as long as is possible.
Change History (10)
This ticket was mentioned in PR #6120 on WordPress/wordpress-develop by @dmsnell.
13 months ago
#1
- Keywords has-patch has-unit-tests added
This ticket was mentioned in Slack in #core-performance by dmsnell. View the logs.
13 months ago
#3
@
13 months ago
- Version changed from trunk to 6.3
Setting version to 6.3 because it was indicated on Slack that this might have been introduced in [55706].
#4
@
12 months ago
- Owner set to dmsnell
- Resolution set to fixed
- Status changed from new to closed
In 57805:
#5
@
12 months ago
- Keywords dev-feedback added
- Milestone changed from Awaiting Review to 6.5
- Resolution fixed deleted
- Status changed from closed to reopened
Re-opening for back-port to 6.5.
Trac ticket: Core-60697
When making repeated updates to a document, the Tag Processor will end
up copying the entire document once for every update. This can lead to
catastrophic behavior in the worse case.
However, when batch-applying updates it's able to copy chunks of the
document in one thread and only end up copying the entire document once
for the entire batch.
Previously the Tag Processor has been eagerly applying updates, but in
this patch it defers applying those updates as long as is possible.