Skip to content

Hard code using the dir entry cache struct (#3994)#3994

Closed
kinto0 wants to merge 1 commit into
facebook:mainfrom
kinto0:export-D109356961
Closed

Hard code using the dir entry cache struct (#3994)#3994
kinto0 wants to merge 1 commit into
facebook:mainfrom
kinto0:export-D109356961

Conversation

@kinto0

@kinto0 kinto0 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary:

Internal results using telemetry show that using directory caching speeds up did open operations and improves language server speed. This experiment has been running for a few weeks now, and things are looking good, so let's just enable this fully

Reviewed By: grievejia

Differential Revision: D109356961

@meta-cla meta-cla Bot added the cla signed label Jun 30, 2026
@meta-codesync

meta-codesync Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

@kinto0 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D109356961.

@grievejia grievejia 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.

Review automatically exported from Phabricator review in Meta.

@github-actions

This comment has been minimized.

Summary:

Internal results using telemetry show that using directory caching speeds up did open operations and improves language server speed. This experiment has been running for a few weeks now, and things are looking good, so let's just enable this fully

Reviewed By: grievejia

Differential Revision: D109356961
@meta-codesync meta-codesync Bot changed the title Hard code using the dir entry cache struct Hard code using the dir entry cache struct (#3994) Jul 1, 2026
@kinto0 kinto0 force-pushed the export-D109356961 branch from 17963de to f37f518 Compare July 1, 2026 22:27
@github-actions github-actions Bot added size/m and removed size/m labels Jul 1, 2026
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

@meta-codesync meta-codesync Bot closed this in babb7c9 Jul 1, 2026
@meta-codesync meta-codesync Bot added the Merged label Jul 1, 2026
@meta-codesync

meta-codesync Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This pull request has been merged in babb7c9.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants