Skip to content

Batch Fixit traversal - 11% improvement - #571

Open
tpolasek wants to merge 1 commit into
Instagram:mainfrom
tpolasek:perf/batch-fixit-traversal
Open

Batch Fixit traversal - 11% improvement#571
tpolasek wants to merge 1 commit into
Instagram:mainfrom
tpolasek:perf/batch-fixit-traversal

Conversation

@tpolasek

Copy link
Copy Markdown

Resolve metadata once for the active rule batch and pre-index visitor dispatch. Collect unused suppressions during the same CST walk and finalize them after report filtering.

Test Plan

Add coverage for inherited visitor discovery and used and unused suppressions.

Resolve metadata once for the active rule batch and pre-index visitor dispatch. Collect unused suppressions during the same CST walk and finalize them after report filtering.

Add coverage for inherited visitor discovery and used and unused suppressions.
@tpolasek
tpolasek requested a review from amyreese as a code owner August 25, 2026 16:42
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 25, 2026

@stroxler stroxler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this one needs a rebase?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants