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.

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

Very unperformanced CaseSensitivePathsPlugin. #6927

Closed
Carduelis opened this issue Apr 7, 2019 · 6 comments
Closed

Very unperformanced CaseSensitivePathsPlugin. #6927

Carduelis opened this issue Apr 7, 2019 · 6 comments

Comments

@Carduelis
Copy link

Carduelis commented Apr 7, 2019

Bug report

Describe the bug

Very slow building time using npm run dev. CaseSensitivePathsPlugin takes a significant time to proceed.

Fast machine Core i7
SMP

Laptop Pentium
SMP

To Reproduce

Steps to reproduce the behavior, please provide code snippets or a repository:

  1. Install SMP to detect the time
  2. Wrap the config with new SpeedMeasurePlugin().wrap(config)
  3. run npm run dev
  4. See the screenshot above

Expected behavior

The first plugin took a lot of time to execute. Should not be a issue in every repo.

System information

  • OS: Windows
  • Version of Next.js: [ 8.0.3]

Additional context

The issue could be caused by aliases, but no time to check.

@Carduelis
Copy link
Author

I've tested this plugin with { debug: true } and found that it's checking all of the modules including node_modules, so it does 2k+ reads every time

[CaseSensitivePathsPlugin] Total filesystem reads: 2566

@timneutkens
Copy link
Member

8.0.4 doesn't have CaseSensitivePathsPlugin for this reason.

@timneutkens
Copy link
Member

Speed measure plugin looks nice btw, hadn't seen it before 💯

@Carduelis
Copy link
Author

Speed measure plugin looks nice btw, hadn't seen it before 💯

Could you include this measure, btw? The thing is when I wrap it, it breaks further work: next.js does not render anything and a browser does not open the app.

So, to measure I need to enable it, then disable to test is app loading ok. I think, the reason is SMP is not a plugin, but a config wrapper

@Carduelis
Copy link
Author

@timneutkens
After installing next 8.0.4
I've caught the SMP output (via ScreenToGif), cause you clear the console output xD.
img

Thanks!

@timneutkens
Copy link
Member

🙏

@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants