Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Prototype "lazy" haste resolver #1289

Open
afoxman opened this issue Mar 20, 2022 · 4 comments
Open

Prototype "lazy" haste resolver #1289

afoxman opened this issue Mar 20, 2022 · 4 comments
Assignees
@tido64
Labels
feature: metro This is related to Metro

Comments

@afoxman
Copy link
Contributor

afoxman commented Mar 20, 2022

Create a prototype "lazy" haste resolver as described in #992.

Run experiments on small, medium and large monorepos and a unirepo, gathering performance data. Measure speed gain/loss (clock time), memory peak, I/O (bytes read/written). Compare normal watchman runs with "lazy" runs.

Report all findings. If viable, develop an engineering plan to implement in rnx-kit.
@afoxman afoxman mentioned this issue Mar 20, 2022
Closed
@afoxman afoxman added this to rnx-kit Mar 20, 2022
@afoxman afoxman self-assigned this Mar 20, 2022
@afoxman afoxman moved this to 🌳 Backlog in rnx-kit Mar 20, 2022
@afoxman
Copy link
Contributor Author

afoxman commented Mar 21, 2022

Related to jestjs/jest#12231

@afoxman
Copy link
Contributor Author

afoxman commented May 12, 2022

Metro recently released their jest-haste-map impl as a proper package: facebook/metro#812

@tido64
Copy link
Member

tido64 commented Jan 12, 2023

As a temporary workaround, we've implemented a way for reducing the scope of the Haste resolver. It's currently implemented as a monkey patch since there is no good way to "configure" this behaviour in Metro. Pending internal testing, we will look at ways to upstream this work: #2044

@tido64 tido64 assigned tido64 and unassigned afoxman Jan 12, 2023
@tido64 tido64 added the feature: metro This is related to Metro label Feb 15, 2023
@kelset
Copy link
Contributor

kelset commented Aug 7, 2023

we want to start Metro without incurring in the cost of Metro's initial crawling

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

@tido64 tido64

Labels
feature: metro This is related to Metro
Projects
rnx-kit
Status: 🌳 Backlog
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tido64 @afoxman @kelset