Skip to content

chore(compiler): improve performance for isolatedModules: false#1558

Merged
ahnpnl merged 7 commits intokulshekhar:25.xfrom
ahnpnl:improve-performance-1
Apr 22, 2020
Merged

chore(compiler): improve performance for isolatedModules: false#1558
ahnpnl merged 7 commits intokulshekhar:25.xfrom
ahnpnl:improve-performance-1

Conversation

@ahnpnl
Copy link
Copy Markdown
Collaborator

@ahnpnl ahnpnl commented Apr 22, 2020

Summary

Improve performance for isolatedModules: false by only creating LanguageService instance with limited amount of initial files.

Because when creating LanguageService instance with fileNames from tsconfig, often this fileNames includes all the files in the project which makes internal TypeScript performs scanning and reading the unnecessary files. This is not optimal because it will create lots of I/O threads which then impact heavily on the performance.

This PR should help #1115 and all related performance issues by somewhere around 30% improvement in speed.

Test plan

Green CI

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

N.A.

@ahnpnl ahnpnl requested a review from kulshekhar as a code owner April 22, 2020 11:13
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 4504

  • 48 of 48 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 92.893%

Totals Coverage Status
Change from base Build 4502: 0.05%
Covered Lines: 1074
Relevant Lines: 1109

💛 - Coveralls

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.

2 participants