Skip to content

perf(request): batch broadcast element deletion - #456

Merged
linkdata merged 1 commit into
mainfrom
fix/batch-broadcast-delete
Sep 29, 2026
Merged

linkdata merged 1 commit into
mainfrom
fix/batch-broadcast-delete

Conversation

@linkdata

Copy link
Copy Markdown
Owner

Summary

  • Delete all elements matched by a broadcast what.Delete in one DeleteElements call before queuing browser Delete frames.
  • Preserve the target snapshot and outbound frames, with a test covering deletion, an unaffected element, and stale frame filtering.
  • Add a benchmark for deleting 100 of 1,000 tagged elements.

Performance

Six runs with benchstat: 163.22 µs/op to 10.73 µs/op (93.43% faster, p=0.002), with unchanged allocations.

Verification

  • JAWS_REQUIRE_NODE=1 go test ./... -count=1
  • JAWS_REQUIRE_NODE=1 go test -race ./... -count=1
  • go vet ./...

Fixes #428

@linkdata
linkdata merged commit 3cc3d78 into main Sep 29, 2026
7 checks passed
@linkdata
linkdata deleted the fix/batch-broadcast-delete branch September 29, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

requestloop: batch broadcast Delete through DeleteElements

1 participant